Fix activation-related race conditions in browser_BrowserWindowTracker.js

RESOLVED FIXED in Firefox 64

Status

()

P1
normal
RESOLVED FIXED
3 months ago
3 months ago

People

(Reporter: Gijs, Assigned: Gijs)

Tracking

Trunk
Firefox 64
Points:
---

Firefox Tracking Flags

(firefox64 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 months ago
With the blank page stuff flipped in bug 1362774, I see Linux-only orange in the BrowserWindowTracker test.

Dão suggested that this has to do with activation events. Waiting for those, as we already do in the captive portal tests, seems to fix things, and can be landed separately, so I'm splitting it out.
(Assignee)

Comment 1

3 months ago
Created attachment 9008538 [details]
Bug 1490819 - wait for windows to be active in browser_BrowserWindowTracker.js, r?dao
Comment on attachment 9008538 [details]
Bug 1490819 - wait for windows to be active in browser_BrowserWindowTracker.js, r?dao

Dão Gottwald [::dao] has approved the revision.
Attachment #9008538 - Flags: review+

Comment 3

3 months ago
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/da39be9798ad
wait for windows to be active in browser_BrowserWindowTracker.js, r=dao

Comment 4

3 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/da39be9798ad
Status: ASSIGNED → RESOLVED
Last Resolved: 3 months ago
status-firefox64: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
You need to log in before you can comment on or make changes to this bug.