Closed Bug 1419579 Opened 4 years ago Closed 4 years ago

background page alert is broken

Categories

(WebExtensions :: Untriaged, defect, P3)

58 Branch
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mixedpuppy, Unassigned)

References

(Blocks 1 open bug)

Details

In moving the test to mochitest-plain I discovered that the background page alert handling is broken in a couple ways.

- devtoolshim is used (does that work on android?)
- opening the browser console fails in oop, first with some errors in devtools then (with a hacky workaround to fix those) fails with a sec-error trying to open a chrome url from moz-ext
- contentWindow.console.log doesn't work

The console log issue and the test will be fixed in bug 1409592, but the test will be left disabled until console opening works.
The console log issue and the test will be fixed in * bug 1417639 *
Depends on: 1417639
Priority: -- → P3
Alert was fixed in bug 1417639, but we should consider whether it is important to uplift.
Flags: needinfo?(amckay)
I don't think its worth uplifting.
Flags: needinfo?(amckay)
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.