Closed Bug 1688474 Opened 3 years ago Closed 3 years ago

`browser` is undefined in Add-on Preferences tab with e10s enabled

Categories

(Thunderbird :: Add-Ons: Extensions API, defect)

defect

Tracking

(thunderbird_esr78 unaffected, thunderbird86+ fixed)

RESOLVED FIXED
87 Branch
Tracking Status
thunderbird_esr78 --- unaffected
thunderbird86 + fixed

People

(Reporter: standard8, Assigned: darktrojan)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

On the latest Thunderbird build, with e10s enabled, if I try to open a preferences tab which is a WebExtension page, then browser is undefined.

If I load a different WebExtension page in a tab via browser.tabs.create, then the browser object is defined.

Regressed by: tb-fission
No longer blocks: tb-fission

That's weird. I'm sure it worked, and also we have a test that should have failed but didn't. And yet, it doesn't work. I have a fix.

It looks like changing the remoteness of a <browser> element in this way breaks something that the "extension-browser-inserted" event sets up. Firing it again seems to work.

Assignee: nobody → geoff
Status: NEW → ASSIGNED
Target Milestone: --- → 87 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/0001e7c85da6
Fix creation of moz-extension page tabs. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED

Comment on attachment 9199105 [details]
Bug 1688474 - Fix creation of moz-extension page tabs. r?mkmelin

[Approval Request Comment]
Regression caused by (bug #): tb-fission
User impact if declined: extension pages in tabs are broken
Testing completed (on c-c, etc.): landed ~12h ago
Risk to taking this patch (and alternatives if risky): a small chance I've not thought of some side effect, but worth it

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

Comment on attachment 9199105 [details]
Bug 1688474 - Fix creation of moz-extension page tabs. r?mkmelin

[Triage Comment]
Approved for beta

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

Attachment

General

Created:
Updated:
Size: