"WebDriver:SwitchToWindow" has to wait for the "activate" and "focus" events before returning
Categories
(Remote Protocol :: Marionette, defect, P1)
Tracking
(firefox68 fixed, firefox69 fixed)
People
(Reporter: whimboo, Assigned: whimboo)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
Attachments
(3 files)
While writing the patch for 1124604 I noticed a case where Marionette fails to correctly focus a window/tab. While the current handles are updated correctly, the focus doesn't change. Which means we operate with the tab in the background. The steps for the test are: * Open a new tab * Switch to this tab * Open a new window * Switch to the tab in the new window * Switch back to the second tab in the first window With the last step the focus doesn't change. For now I will exclude this specific test from my patch on bug 1124604.
|
||
Updated•7 years ago
|
|
Assignee | |
Comment 1•6 years ago
|
||
This is actually a race because the command doesn't wait until the target window has focus and is activated. My patch on bug 1504756 will add a couple of helpers which will make those checks easier to perform.
|
|
Assignee | |
Comment 2•5 years ago
|
||
To remove the race condition which can result in intermittent failures we should get this bug fixed. Also because it has visibility through geckodriver.
|
|
Assignee | |
Comment 3•5 years ago
|
||
|
|
Assignee | |
Comment 4•5 years ago
|
||
|
|
Assignee | |
Comment 5•5 years ago
|
||
Selecting a different tab is asynchronous, and as such Marionette has
to wait for the "TabSelect" event to be emitted. Further the chrome
window also needs to be focused.
Depends on D33689
|
|
Assignee | |
Comment 6•5 years ago
|
||
Depends on D33690
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e1387e00703d [marionette] Make use of "WebDriver:NewWindow" in firefox-puppeteer. r=webdriver-reviewers,ato https://hg.mozilla.org/integration/autoland/rev/252fc1ac18cc [marionette] Fix "WebDriver:SwitchToWindow" for tab and window focus switching. r=webdriver-reviewers,ato https://hg.mozilla.org/integration/autoland/rev/5dc349bb1690 [marionette] Simplify focus handling for window classes in firefox-puppeteer. r=webdriver-reviewers,ato
|
||
Comment 8•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/e1387e00703d
https://hg.mozilla.org/mozilla-central/rev/252fc1ac18cc
https://hg.mozilla.org/mozilla-central/rev/5dc349bb1690
|
|
Assignee | |
Comment 9•5 years ago
|
||
I would like to keep it on the radar for the Firefox 68 release, given that it massively improves the stability and reduces the risk of race conditions. If there are no regressions in the next couple of days, lets get it uplifted to beta.
|
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 10•5 years ago
|
||
I haven't seen any regressions from landing this patch on mozilla-central. To improve the reliability of Marionette and external tools like Selenium using geckodriver I would like to see this test-only patch uplifted to beta for the ESR release. Thanks.
|
||
Comment 11•5 years ago
|
||
| bugherder uplift | ||
https://hg.mozilla.org/releases/mozilla-beta/rev/dde54edc86e1
https://hg.mozilla.org/releases/mozilla-beta/rev/1e1c5ad53cfc
https://hg.mozilla.org/releases/mozilla-beta/rev/992cbc55ecf7
|
||
Updated•1 year ago
|
Description
•