576458 - storage/test/unit/test_js_helpers.js Object.getOwnPropertyName-using tests are wrong

Status: NEW

(Toolkit :: Storage, defect, P3)

Description

[Jeff Walden [:Waldo]]

If I uncomment the tests there, now that Object.getOwnPropertyNames is implemented in the TraceMonkey tree, they fail.  The first call returns ["a", "b", "c"], as does the second one, so the length-checking arithmetic at the end of each of the two tests fails.  I don't see why that arithmetic would be correct (I had thought these objects were to appear to be objects whose properties were the various "foo" for :foo things, but this isn't what the test expects, and there's not enough detail in the test for me to tell what it really wants), so I've removed the Object.getOwnPropertyNames check and left the test otherwise commented out.