Closed
Bug 1288339
Opened 8 years ago
Closed 7 years ago
Before interacting with a new chrome window ensure that readyState is complete
Categories
(Remote Protocol :: Marionette, defect, P3)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1311041
People
(Reporter: whimboo, Unassigned)
References
(Blocks 1 open bug)
Details
As seen on bug 1287723 we do not always wait for the readyState to be complete before starting to interact with a new chrome window. As result we run into race conditions which can cause failures in attaching our listeners.
Similar to new content windows we should make sure to wait for the window to be completely loaded. For firefox-puppeteer we already have a workaround in place which works pretty well:
https://dxr.mozilla.org/mozilla-central/source/testing/puppeteer/firefox/firefox_puppeteer/ui/windows.py#151
To fix this bug I would propose a solution in marionette-server, especially in driver.js. From what I can see right now `switchToWindow()` should be the best place for it.
Andreas, what do you think? Is that assumption right or would you propose a different location for that check?
Flags: needinfo?
Reporter | ||
Updated•8 years ago
|
Flags: needinfo?
Reporter | ||
Updated•8 years ago
|
Flags: needinfo?(ato)
Reporter | ||
Comment 1•8 years ago
|
||
As noticed on bug 1287723 the broken load behavior might only happen for erroneous chrome window URLs. It would need some further investigation. Removing ni? request from Andreas for now.
Flags: needinfo?(ato)
Updated•7 years ago
|
Priority: -- → P3
Comment 2•7 years ago
|
||
This will be resolved by https://bugzil.la/marionette-window-tracking.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Updated•2 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•