Closed Bug 1690467 Opened 2 years ago Closed 2 years ago

Date/time picker broken


(Thunderbird :: General, defect)



(thunderbird_esr78 unaffected, thunderbird86 fixed, thunderbird87 affected)

87 Branch
Tracking Status
thunderbird_esr78 --- unaffected
thunderbird86 --- fixed
thunderbird87 --- affected


(Reporter: darktrojan, Assigned: darktrojan)




(Keywords: regression)


(1 file)

The introduction of a gBrowser in bug 810814 bug 810815 has broken the date/time picker, which was relying on the non-existence of gBrowser to work. Gah.

Regressed by: 810815
No longer regressed by: 810814

It seems like gBrowser is no longer required by pdf.js, so I can remove it.

Assignee: nobody → geoff

I just tried this locally, and it gives errors when you load a pdf:

TypeError: can't access property "getTabForBrowser", tabbrowser is null
_hookupEventListeners resource://pdf.js/PdfjsParent.jsm:232
    _addEventListener resource://pdf.js/PdfjsParent.jsm:222
    receiveMessage resource://pdf.js/PdfjsParent.jsm:84

These seem related to the find bar - they don't seem to stop it working, but I would be concerned about removing them.

You could possibly see if aliasing tabbrowser to tabmail might work.

There's also quite a bit of toolkit items that depend on gBrowser:

Not saying you can't remove it, but I think this needs careful examination.

The presence of gBrowser confuses the DateTimePicker actor. Since pdf.js has now been updated to run without gBrowser (instead using browser.getTabBrowser) this patch removes it.

Pushed by
Fix date/time picker by removing gBrowser variable. r=mkmelin

Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 87 Branch

Comment on attachment 9201029 [details]
Bug 1690467 - Fix date/time picker by removing gBrowser variable. r?mkmelin

[Approval Request Comment]
Regression caused by (bug #): bug 810815
User impact if declined: HTML <input type=date> won't work
Testing completed (on c-c, etc.): on c-c with a new test since yesterday
Risk to taking this patch (and alternatives if risky): maybe, as Mark mentioned we're removing a variable toolkit uses, but it's only been there months and we've been careful to make sure stuff works without it in the past, so I think it's fine.

Attachment #9201029 - Flags: approval-comm-beta?

Comment on attachment 9201029 [details]
Bug 1690467 - Fix date/time picker by removing gBrowser variable. r?mkmelin

[Triage Comment]
Approved for beta

Attachment #9201029 - Flags: approval-comm-beta? → approval-comm-beta+
You need to log in before you can comment on or make changes to this bug.