NEW 174242
[GTK] Layout test media/media-fullscreen-pause-inline.html is timing out
https://bugs.webkit.org/show_bug.cgi?id=174242
Summary [GTK] Layout test media/media-fullscreen-pause-inline.html is timing out
Michael Catanzaro
Reported 2017-07-06 22:38:41 PDT
Layout test media/media-fullscreen-pause-inline.html is timing out on the GTK port since it was added in r218847 "Disable m_temporarilyAllowingInlinePlaybackAfterFullscreen on pause". The new test is skipped on mac-wk2, but there's no explanation why. Hey Jeremy, should it be skipped on more ports? If so, why? I'll add a timeout expectation for GTK.
Attachments
Ms2ger (he/him; ⌚ UTC+1/+2)
Comment 1 2017-08-07 09:03:59 PDT
Looks like the issue is that ChromeClient::supportsVideoFullscreen is not overridden, so HTMLMediaElement::exitFullscreen() never fires the "webkitendfullscreen" event needed to continue the test. Changing ChromeClient::supportsVideoFullscreen to return true makes the test pass, but I don't know what other effects it would have. Jeremy, thoughts?
Philippe Normand
Comment 2 2018-01-11 02:38:51 PST
I believe this test works only on iOS or when the VIDEO_PRESENTATION_MODE feature is enabled. Is that right, Eric? I think GTK should skip it.
Eric Carlson
Comment 3 2018-01-11 08:50:16 PST
(In reply to Philippe Normand from comment #2) > I believe this test works only on iOS or when the VIDEO_PRESENTATION_MODE > feature is enabled. Is that right, Eric? > This doesn't test presentation mode, it tests a mode where although inline playback isn't supported directly, playback does continue inline if a user exits from fullscreen without pausing a video. > I think GTK should skip it. > That seems like the right solution since GTK doesn't support this unusual mode. Sorry about this, the test should have been skipped on other ports when it was added.
Philippe Normand
Comment 4 2018-01-15 07:49:46 PST
Note You need to log in before you can comment on or make changes to this bug.