Closed
Bug 1380513
Opened 7 years ago
Closed 7 years ago
Intermittent test_window_maximize.py TestWindowMaximize.test_maximize_twice_restores | AssertionError: 1500 != 1600
Categories
(Testing :: Marionette Client and Harness, defect)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: intermittent-bug-filer, Unassigned)
References
Details
(Keywords: intermittent-failure)
Comment 1•7 years ago
|
||
Sounds similar to what we see on bug 1373879.
Comment 2•7 years ago
|
||
Hm, whereby this other bug was fixed. Andreas, any idea?
Flags: needinfo?(ato)
Comment hidden (Intermittent Failures Robot) |
Comment 4•7 years ago
|
||
My guess is that we need to use some throttling
like we do for WebDriver:SetWindowRect in
https://searchfox.org/mozilla-central/rev/01d27fdd3946f7210da91b18fcccca01d7324fe2/testing/marionette/driver.js#1360-1409.
This resize/repositioning code could probably be repurposed
directly.
I do find it strange that the stress test for Maximize Window
that follows doesn’t exhibit the this problem, but it could be
a super-rare intermittent. OrangeFactor seems to indicate it is
pretty rare.
Flags: needinfo?(ato)
Comment 5•7 years ago
|
||
I can look into this after
https://bugzilla.mozilla.org/show_bug.cgi?id=1311041, as it fits
naturally on some of the new browsing context abstractions that
I’m working on there. Judging by how infrequent this intermittent
is, it’s probably worth waiting for.
Comment 6•7 years ago
|
||
(In reply to Andreas Tolfsen ‹:ato› from comment #4)
> I do find it strange that the stress test for Maximize Window
> that follows doesn’t exhibit the this problem, but it could be
> a super-rare intermittent. OrangeFactor seems to indicate it is
> pretty rare.
And it appears it does have the same problem, as was reported in
https://bugzilla.mozilla.org/show_bug.cgi?id=1380309.
I’ve filed https://bugzilla.mozilla.org/show_bug.cgi?id=1381876 to
ensure the Maximize Window command becomes synchronous.
Updated•7 years ago
|
Comment hidden (Intermittent Failures Robot) |
Comment 8•7 years ago
|
||
This test has been rewritten and we dont support that feature anymore so closing
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Updated•2 years ago
|
Product: Testing → Remote Protocol
Comment 9•2 years ago
|
||
Moving bug to Testing::Marionette Client and Harness component per bug 1815831.
Component: Marionette → Marionette Client and Harness
Product: Remote Protocol → Testing
You need to log in
before you can comment on or make changes to this bug.
Description
•