is both superfluous and unwise. You declare allTextFrames, initially assign it the value of the Array constructor, as if it were an array, and then you assign it to a collection.

A collection is not an array. So now you use the same variable to represent two different types. This is acceptable in JavaScript, but bad style, because it promotes confusion about what type things are.