WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Add attachment
proposed patch, testcase, etc.
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
Test skipped in
https://trac.webkit.org/r226949
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug