1589508 - Fix three xpinstall/browser_enabled* tests with fission

Status: RESOLVED FIXED

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

Description

[Tomislav Jovanovic :zombie]

[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.

Comment 1

[Tomislav Jovanovic :zombie]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=7b3e9d94524e108f1295e4e024b4708ea120c2f6

Comment 2

[Tomislav Jovanovic :zombie]

Attachment: Bug 1589508 - Fix three xpinstall/browser_enabled* tests

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

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1640f6921451
Fix three xpinstall/browser_enabled* tests r=mixedpuppy

Comment 4

[Dorel Luca [:dluca]]

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

Comment 5

[Alex Cornestean]

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

Comment 6

[Chris Peterson [:cpeterson]]

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