Closed Bug 1690141 Opened 3 years ago Closed 3 years ago

Popup window not closed anymore

Categories

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

Thunderbird 86
Desktop
Windows 10
defect

Tracking

(thunderbird_esr78 unaffected)

RESOLVED FIXED
89 Branch
Tracking Status
thunderbird_esr78 --- unaffected

People

(Reporter: k-l-p, Assigned: TbSync)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:85.0) Gecko/20100101 Firefox/85.0

Steps to reproduce:

Run Add-on "Textmarker" Version 5.0.1
In compose window click "Ctrl"+Mark-Button to get colour change popup.
Click on a colour.

Actual results:

Colour is taken, sound is played, but popup window is not closed.

(If browser.tabs.remote.autostart is set to false, the popup windows is closed as intended.)

Expected results:

Popup window should close automatically after colour picking.

Code in popup.js:
window.setTimeout(closewin, 700);
function closewin() {
window.close();
}

OS: Unspecified → Windows 10
Hardware: Unspecified → Desktop
Flags: needinfo?(john)
Component: Untriaged → Add-Ons: Extensions API
Regressed by: tb-fission
Keywords: regression

Yes, it is related to e10s being enabled. If you toggle the browser.tabs.remote.autostart preference and restart Thunderbird, it works again.

Flags: needinfo?(john)
Status: UNCONFIRMED → NEW
Ever confirmed: true

In 86.0b2 with browser.tabs.remote.autostart set to true, popup window closes as intended - problem seems to be resolved.

Do we know which bug could have fixed this?

I'm sorry, maybe I made a mistake when toggling the pref ...
In any case, in 86.0b3 the problem still exists (or exists again).

Just tested 87.0b1: the problem still exists.

Assignee: nobody → john
Status: NEW → ASSIGNED

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/75e8be79df16
Fix window.close() in extension popup windows. r=darktrojan DONTBUILD

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: