Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(179)

Unified Diff: LayoutTests/media/video-play-require-user-gesture.html

Issue 191273002: Revert "Remove the HTMLVideoElement-specific prefixed fullscreen API" (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/media/video-play-require-user-gesture.html
diff --git a/LayoutTests/media/video-play-require-user-gesture.html b/LayoutTests/media/video-play-require-user-gesture.html
index 0e936071238ab2a2c6c1cd2d00fe44c120157d38..55ad82e4cb16b380a314d7199ca153b522929cdc 100644
--- a/LayoutTests/media/video-play-require-user-gesture.html
+++ b/LayoutTests/media/video-play-require-user-gesture.html
@@ -1,6 +1,6 @@
<html>
<head>
- <title>Test that video play does not work unless a user gesture is involved in playing a video</title>
+ <title>Test that video play, pause and enterfullscreen does not work unless a user gesture is involved in playing a video</title>
<script src=media-controls.js></script>
<script src=media-file.js></script>
<script src=video-test.js></script>
@@ -41,6 +41,14 @@
function pause()
{
testExpected("video.paused", true);
+ // Now video.webkitEnterFullScreen() should no longer throw any exception.
+ // However, the video element may not always enter fullscreen. For example,
+ // chromium uses fullscreen API which still requires user gesture
+ try {
+ run("video.webkitEnterFullScreen()");
+ } catch(ex) {
+ failTest("video.webkitEnterFullScreen() still requires user gesture.");
+ }
endTest();
}
@@ -49,6 +57,7 @@
consoleWrite("");
consoleWrite("* No user gesture initiated");
run("video.play()");
+ testDOMException("video.webkitEnterFullScreen()", "DOMException.INVALID_STATE_ERR");
testExpected("video.paused", true);
consoleWrite("");
@@ -68,7 +77,7 @@
</head>
<body onload="start()">
- <p>Test that video play() does not work unless a user clicked on the play button.</p>
+ <p>Test that video play(), pause() and webkitEnterFullScreen() should not work unless a user clicked on the play button.</p>
<video controls></video>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698