796805 - Permanent orange: TEST-UNEXPECTED-FAIL | test_iteratorUtils.js | TypeError: childArray is null

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[Mark Banner (:standard8)]

This just appeared in comm-central, note that we've had a similar issue before with bug 751873.

There's no obvious changesets jumping out at me, so I have bisect running.

>>>>>>>

TEST-INFO | (xpcshell/head.js) | test 1 pending

TEST-PASS | /home/cltbld/talos-slave/test/build/xpcshell/tests/mailnews/base/test/unit/test_iteratorUtils.js | [test_toArray_NodeList : 35] rootnode == rootnode

TEST-PASS | /home/cltbld/talos-slave/test/build/xpcshell/tests/mailnews/base/test/unit/test_iteratorUtils.js | [test_toArray_NodeList : 39] true == true

TEST-UNEXPECTED-FAIL | /home/cltbld/talos-slave/test/build/xpcshell/tests/mailnews/base/test/unit/test_iteratorUtils.js:41 | TypeError: childArray is null - See following stack:
JS frame :: test_toArray_NodeList@/home/cltbld/talos-slave/test/build/xpcshell/tests/mailnews/base/test/unit/test_iteratorUtils.js:41
JS frame :: run_test@/home/cltbld/talos-slave/test/build/xpcshell/tests/mailnews/base/test/unit/test_iteratorUtils.js:90
JS frame :: _execute_test@/home/cltbld/talos-slave/test/build/xpcshell/head.js:315
JS frame :: @-e:1
<<<<<<<

Comment 1

[Florian Quèze [:florian]]

Attachment: Fix

I added a dump at http://mxr.mozilla.org/comm-central/source/mailnews/base/util/iteratorUtils.jsm#38 and saw that aObj.constructor.toString() contains "function NodeList() {\n [native code]\n}".

Comment 2

[Florian Quèze [:florian]]

Attachment: Fix

The file I actually meant to attach this time (same patch without syntax error).

Comment 3

[Florian Quèze [:florian]]

https://hg.mozilla.org/comm-central/rev/86bb28178965