Closed Bug 1261509 Opened 4 years ago Closed 4 years ago

JavaScript error: chrome://global/content/bindings/textbox.xml, line 174: TypeError: this.inputField is undefined

Categories

(Toolkit :: XUL Widgets, defect)

48 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox46 --- wontfix
firefox47 --- wontfix
firefox48 --- fixed

People

(Reporter: kats, Assigned: kats)

References

Details

Attachments

(1 file)

While converting a bunch of tests from mochitest-chrome to mochitest-plain for e10s compatibility in bug 1261158, I ran into this error. One of the tests (image/test/mochitest/test_xultree_animation.xhtml) creates a xul:tree, which contains a xul:textbox. The test runs fine and passes, but then when the browser shuts down, the xul:textbox is destroyed, and the destructor spits out this error:

JavaScript error: chrome://global/content/bindings/textbox.xml, line 174: TypeError: this.inputField is undefined

The mochitest harness (SimpleTest.js) catches this and it results in a failure, sort of indirectly as described at https://bugzilla.mozilla.org/show_bug.cgi?id=1261158#c19.

I discussed this with Gijs on IRC [1] and he suggested adding a null guard here, although there is likely a deeper issue lurking somewhere with the XBL shutdown sequence.

[1] http://logs.glob.uno/?c=mozilla%23developers&s=1%20Apr%202016&e=1%20Apr%202016#c1404796
Attached patch Add guardSplinter Review
Attachment #8737383 - Flags: review?(enndeakin)
Assignee: nobody → bugmail.mozilla
Version: 46 Branch → 48 Branch
Attachment #8737383 - Flags: review?(enndeakin) → review+
https://hg.mozilla.org/mozilla-central/rev/dda9f810dd7a
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Not sure this is worth uplifting.
You need to log in before you can comment on or make changes to this bug.