Closed Bug 1410292 Opened 2 years ago Closed 2 years ago

[DateTimeInput] TypeError: this.mInputElement is null

Categories

(Core :: Layout: Form Controls, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox57 --- wontfix
firefox58 --- fixed

People

(Reporter: jessica, Assigned: jessica)

Details

Attachments

(1 file)

(In reply to Jessica Jong [:jessica] from comment #0)
> Agrr, happens in the <destructor> in datetimebox.xml [1], missed this when
> fixing bug 137111.
         ^^^^^^^^^^^

I meant bug 1371111 :/

> 
> 
> [1]
> http://searchfox.org/mozilla-central/rev/
> 1c4da216e00ac95b38a3f236e010b31cdfaae03b/toolkit/content/widgets/datetimebox.
> xml#1256
Priority: -- → P3
Attached patch patch.Splinter Review
Hi Mike, can you take a look at this tiny patch? Thanks.
Attachment #8922253 - Flags: review?(mconley)
Comment on attachment 8922253 [details] [diff] [review]
patch.

Review of attachment 8922253 [details] [diff] [review]:
-----------------------------------------------------------------

Ooof, whoops. :/ Thanks for fixing!
Attachment #8922253 - Flags: review?(mconley) → review+
Pushed by jjong@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5c77250f74a4
Clear mInputElement after accessing it. r=mconley
https://hg.mozilla.org/mozilla-central/rev/5c77250f74a4
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.