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

Status: RESOLVED FIXED

(Toolkit :: UI Widgets, defect)

Description

[Kartikaya Gupta (email:[e-mail])]

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

Comment 1

[Kartikaya Gupta (email:[e-mail])]

Attachment: Add guard

Comment 2

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/dda9f810dd7a

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/dda9f810dd7a

Comment 4

[Kartikaya Gupta (email:[e-mail])]

Not sure this is worth uplifting.