Closed Bug 1367227 Opened 5 years ago Closed 5 years ago

Intermittent TestWindowMaximize.test_maximize | IOError: Process killed because the connection to Marionette server is lost. Check gecko.log for errors (Reason: Connection timed out after 360.0s)


(Testing :: Marionette, defect)

Version 3
Not set


(Not tracked)



(Reporter: intermittent-bug-filer, Assigned: bdahl)



(Whiteboard: [stockwell disabled])


(3 files, 1 obsolete file)

Comment on attachment 8870571 [details]
Bug 1367227: Disable Marionette window maximize tests in headless mode;
Attachment #8870571 - Flags: review?(bdahl) → review+
Pushed by
Disable Marionette window maximize tests in headless mode; r=bdahl
Pushed by
Disable Marionette window rect tests in headless mode. r=bdahl
The oranges are gone now, but let's leave this bug open for the eventual fixing & re-enabling.
Assignee: nobody → bdahl
Keywords: leave-open
Whiteboard: [stockwell disabled]
During MakeFullScreen, headless wasn't triggering a size mode and full screen changed event which caused marionette to hang waiting for a resize event. I tried to match more of the behavior of the linux and windows backend with the new code.
Attachment #8872000 - Flags: review?(jmuizelaar)
As mentioned on IRC, the current code doesn't actually emulate what  "View > Enter Full Screen" does and it will cause the browser to immediately try to exit fullscreen mode because it triggers the event on a chrome window and it's expected to be on a content window (follow aBrowser in [1]). This doesn't seem to be an issue on various platforms because the exit is ignored while the full screen transition is in progress. The new way does was browser.js does [2].

Attachment #8872004 - Flags: review?(ato)
And a try run (still waiting on MacOS, but looks good elsewhere):
Blocks: 1356681
Attachment #8872000 - Flags: review?(jmuizelaar) → review+
Attachment #8872004 - Flags: review?(ato) → review+
Looks like I should have waited on those osx jobs... It appears that some of the resize events on mac come before the fullscreen transition is complete. I've switched it to now use sizemodechange, as that seems like the more reliable way.
Attachment #8872840 - Flags: review?(ato)
Attachment #8872004 - Attachment is obsolete: true
Comment on attachment 8872840 [details] [diff] [review]
Part 2 - Trigger fullscreen the same way Firefox does.

Review of attachment 8872840 [details] [diff] [review]:

r+, push when confident.
Attachment #8872840 - Flags: review?(ato) → review+
Pushed by
Part 1 - Match headless fullscreen behavior to other widget backends. r=jrmuizel
Part 2 - Trigger fullscreen the same way Firefox does. r=ato
Oranges are gone.
Keywords: leave-open
Closed: 5 years ago
Resolution: --- → FIXED
No known fix -> WFM.
You need to log in before you can comment on or make changes to this bug.