background page alert is broken

NEW
Unassigned

Status

()

Toolkit
WebExtensions: Untriaged
P3
normal
2 months ago
6 days ago

People

(Reporter: mixedpuppy, Unassigned)

Tracking

(Depends on: 1 bug)

58 Branch
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

2 months 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

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

Updated

2 months ago
Priority: -- → P3
(Reporter)

Comment 2

12 days ago
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)
You need to log in before you can comment on or make changes to this bug.