Closed Bug 1589508 Opened 2 years ago Closed 2 years ago

Fix three xpinstall/browser_enabled* tests with fission

Categories

(Toolkit :: Add-ons Manager, defect, P2)

Product:
Toolkit
Component:
Add-ons Manager
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla71
Project Flags:
Fission Milestone M4
Tracking Flags:
Tracking Status
firefox71 --- fixed

People

(Reporter: zombie, Assigned: zombie)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Bug 1589508 - Fix three xpinstall/browser_enabled* tests
47 bytes, text/x-phabricator-request
Details | Review

[browser_enabled.js]
[browser_enabled2.js]
[browser_enabled3.js]

Actual tests only start after the initial page is loaded, and don't care what was previously in the same tab. Under fission, starting with about:blank involves a process switch, which ContentTask isn't equipped to handle, so just need to avoid that.

Component: General → Add-ons Manager
Product: WebExtensions → Toolkit

[browser_enabled.js]
[browser_enabled2.js]
[browser_enabled3.js]

Pushed by tjovanovic@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1640f6921451
Fix three xpinstall/browser_enabled* tests r=mixedpuppy

https://hg.mozilla.org/mozilla-central/rev/1640f6921451

Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox71: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla71

Setting the qe-verify- flag as this is a test related fix.

Flags: qe-verify-
Blocks: 1589885

Retroactively moving fixed bugs whose summaries mention "Fission" (or other Fission-related keywords) but are not assigned to a Fission Milestone to an appropriate Fission Milestone.

This will generate a lot of bugmail, so you can filter your bugmail for the following UUID and delete them en masse:

0ee3c76a-bc79-4eb2-8d12-05dc0b68e732

Fission Milestone: --- → M4
You need to log in before you can comment on or make changes to this bug.