WindowManagerMixin:open_chrome_window() registers focus event listeners too late

RESOLVED FIXED in Firefox 66

Status

enhancement
P2
normal
RESOLVED FIXED
5 months ago
5 months ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

Trunk
mozilla66
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox66 fixed)

Details

Attachments

(1 attachment)

The JS code in that method registers the event listeners for focus and activate after calling win.focus(). It means we have a race conditions for tests trying to open chrome windows.

The focus promise has to be setup before calling focus() on the window.

Blocks: 1521897
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Priority: -- → P2

Comment 3

5 months ago
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9762341e99f9
[marionette] Register event listeners before focusing the new window in WindowManagerMixin.open_chrome_window(). r=ato

Comment 4

5 months ago
bugherder
Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in before you can comment on or make changes to this bug.