Replaced two for..in style for-loops (in ltrTreeGraph.js) that contained a nested splice call with a safer for-loop.  The for..in style with a nested splice call can cause the array being iterated over to change length, which causes the for..in to change its indexing range.  Ultimately, resulting skipped array elements.  Now, using safe setup where the array is iterated starting with the end element and decrementing to the beginning so that removing the 'current' index doesn't affect the indexing.
Replaced two for..in style for-loops (in ltrTreeGraph.js) that contained a nested splice call with a safer for-loop.  The for..in style with a nested splice call can cause the array being iterated over to change length, which causes the for..in to change its indexing range.  Ultimately, resulting skipped array elements.  Now, using safe setup where the array is iterated starting with the end element and decrementing to the beginning so that removing the 'current' index doesn't affect the indexing.

Netflix/vizceral