Closed Bug 1780977 Opened 2 years ago Closed 2 years ago

alert/confirm/prompt do not show in Web extension options pages

Categories

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

Tracking

(thunderbird_esr102+ fixed, thunderbird107 affected)

RESOLVED FIXED
108 Branch
Tracking Status
thunderbird_esr102 + fixed
thunderbird107 --- affected

People

(Reporter: tdulcet, Assigned: TbSync)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

alert(), confirm() and prompt() do not show in Thunderbird options pages. They work fine in Firefox.

I tested in the latest Thunderbird 91 ESR and 102/103 beta. Using mozregression, it looks like it was regressed by something related to bug 1646648. I get this error in the Error console:

NS_ERROR_ILLEGAL_VALUE: Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsIWindowWatcher.openWindow] PromptParent.jsm:161
Keywords: regression
Regressed by: tb-fission
Severity: -- → S3
Priority: -- → P3
Assignee: nobody → john
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Target Milestone: --- → 108 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/8b330f7342ec
Check window type before using it as parent window in Services.ww.openWindow(). r=mkmelin

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

Comment on attachment 9300321 [details]
Bug 1780977 - Check window type before using it as parent window in Services.ww.openWindow(). r=mkmelin

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined:
alert/confirm/prompt do not show in Web extension options pages

Testing completed (on c-c, etc.):
On Daily for more than 20 days, on beta since merge day (14.11)

Risk to taking this patch (and alternatives if risky):
Low.

Attachment #9300321 - Flags: approval-comm-esr102?

Comment on attachment 9300321 [details]
Bug 1780977 - Check window type before using it as parent window in Services.ww.openWindow(). r=mkmelin

[Triage Comment]
Approved for esr102

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

Attachment

General

Created:
Updated:
Size: