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)

Version 3
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: intermittent-bug-filer, Unassigned)

References

Details

(Keywords: intermittent-failure)

Sounds similar to what we see on bug 1373879.
See Also: → 1373879
Hm, whereby this other bug was fixed. Andreas, any idea?
Flags: needinfo?(ato)
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)
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.
Assignee: nobody → ato
Status: NEW → ASSIGNED
Depends on: 1381876
(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.
Assignee: ato → nobody
Status: ASSIGNED → NEW
No longer depends on: marionette-window-tracking
This test has been rewritten and we dont support that feature anymore so closing
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Product: Testing → Remote Protocol
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.