"WebDriver:NewWindow" causes race conditions because it doesn't wait until the initial "about:blank" has been finished loading
Categories
(Remote Protocol :: Marionette, defect, P1)
Tracking
(Fission Milestone:M7, firefox84 fixed)
Tracking | Status | |
---|---|---|
firefox84 | --- | fixed |
People
(Reporter: whimboo, Assigned: jdescottes)
References
Details
(Whiteboard: [marionette-fission-mvp])
Attachments
(2 files, 2 obsolete files)
As discovered on bug 1673851 comment 11 the WebDriver:NewWindow
command currently doesn't await the load of about:blank
, and as such creates race conditions with the next command(s) due to an unexpected page load happening.
Because the WebDriver spec doesn't mention it yet, I filed https://github.com/w3c/webdriver/issues/1560.
Nevertheless we have to get it implemented in Marionette, and I don't think we have to wait for the spec to be updated.
Reporter | ||
Updated•4 years ago
|
Reporter | ||
Comment 1•4 years ago
|
||
Moving to Julian because he already has patch in the works.
Comment 2•4 years ago
|
||
Tracking marionette-fission-mvp bugs for Fission Beta milestone (M7).
Assignee | ||
Comment 3•4 years ago
|
||
Expose the boilerplate code to register the MarionetteEventsActor to later reuse it from driver.js
Assignee | ||
Comment 4•4 years ago
|
||
Depends on D96117
Assignee | ||
Comment 5•4 years ago
|
||
Depends on D96118
Updated•4 years ago
|
Updated•4 years ago
|
Reporter | ||
Comment 6•4 years ago
|
||
Note that my patch on bug 1675248 caused perma failures for the wdspec new window tests. Maybe it's a good way to verify that this change works as expected.
Updated•4 years ago
|
Assignee | ||
Comment 7•4 years ago
|
||
Depends on D96119
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 9•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/bb6072f5d067
https://hg.mozilla.org/mozilla-central/rev/0206b72af90a
Updated•2 years ago
|
Description
•