Closed Bug 1594276 Opened 5 years ago Closed 5 years ago

Fix browser_keepStateAcrossTabSwitches.js to work with Fission

Categories

(Firefox :: Address Bar, task, P1)

task
Points:
2

Tracking

()

RESOLVED FIXED
Firefox 72
Iteration:
72.2 - Nov 4 - 17
Fission Milestone M4
Tracking Status
firefox72 --- fixed

People

(Reporter: adw, Assigned: adw)

Details

Attachments

(1 file)

browser_keepStateAcrossTabSwitches.js is skipped on Fission.

The test hangs in the first task on await errorPageLoaded. aboutNetError.js does fire the AboutNetErrorLoad event that BrowserTestUtils is waiting for, and BrowserTestUtils does add its listener before the event is fired, but the event never reaches the listener for some reason. Changing the initial page in the test from about:blank to an actual page and waiting for it to load fixes it, but I don't know why. The test doesn't depend on the initial page, so this should be OK.

Pushed by dwillcoxon@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5aae86b2b1ee Fix browser_keepStateAcrossTabSwitches.js to work with Fission r=mak
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 72

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.

Attachment

General

Created:
Updated:
Size: