background page alert is broken

RESOLVED FIXED

Status

P3
normal
RESOLVED FIXED
a year ago
4 months ago

People

(Reporter: mixedpuppy, Unassigned)

Tracking

(Blocks: 1 bug)

58 Branch
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

a year ago
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.
(Reporter)

Comment 1

a year ago
The console log issue and the test will be fixed in * bug 1417639 *
Depends on: 1417639

Updated

a year ago
Priority: -- → P3
(Reporter)

Comment 2

10 months ago
Alert was fixed in bug 1417639, but we should consider whether it is important to uplift.
Flags: needinfo?(amckay)

Comment 3

10 months ago
I don't think its worth uplifting.
Flags: needinfo?(amckay)
(Reporter)

Updated

6 months ago
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → FIXED

Updated

5 months ago
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.