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

Unified Diff: third_party/WebKit/LayoutTests/media/autoplay-when-visible.html

Issue 2051253002: Start autoplay muted videos with autoplay attribute when they are visible. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review comments Created 4 years, 6 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: third_party/WebKit/LayoutTests/media/autoplay-when-visible.html
diff --git a/third_party/WebKit/LayoutTests/media/autoplay-when-visible.html b/third_party/WebKit/LayoutTests/media/autoplay-when-visible.html
new file mode 100644
index 0000000000000000000000000000000000000000..a97807eb2ed1af758f8e8414e0c8b6dad56b87bc
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/media/autoplay-when-visible.html
@@ -0,0 +1,102 @@
+<!DOCTYPE html>
+<html>
+<head>
esprehn 2016/06/21 21:28:41 ditto
+<title>Test behaviour of autoplay muted videos with regards to visibility</title>
+</head>
+<body>
esprehn 2016/06/21 21:28:41 put the body after the media-file.js, right before
mlamouri (slow - plz ping) 2016/06/21 21:32:11 What's the intent?
esprehn 2016/06/21 21:36:40 It's the most minimal thing needed to make your te
+<script src="../resources/testharness.js"></script>
+<script src="../resources/testharnessreport.js"></script>
+<script src="media-file.js"></script>
+<script>
+ window.internals.settings.setMediaPlaybackRequiresUserGesture(true);
+ window.internals.runtimeFlags.autoplayMutedVideosEnabled = true;
+
+ async_test(function(t) {
+ // Create a video off screen.
+ {
+ var video = document.createElement('video');
+ video.id = 'offscreen';
+ video.src = findMediaFile('video', 'content/test');
+ video.muted = true;
+ video.autoplay = true;
+ video.loop = true;
+ video.style.position = 'absolute';
+ video.style.top = '-10000px';
+ document.body.appendChild(video);
+ }
+
+ // Create a video in screen.
+ {
+ var video = document.createElement('video');
+ video.id = 'inscreen';
+ video.src = findMediaFile('video', 'content/test');
+ video.muted = true;
+ video.autoplay = true;
+ video.loop = true;
+ document.body.appendChild(video);
+ }
+
+ // Create an audio in screen.
+ {
+ var audio = document.createElement('audio');
+ audio.src = findMediaFile('audio', 'content/test');
+ audio.muted = true;
+ audio.autoplay = true;
+ audio.loop = true;
+ audio.controls = true;
+ document.body.appendChild(audio);
+ }
+
+ // Create offscreen video without autoplay.
+ {
+ var video = document.createElement('video');
+ video.id = 'offscreen-no-autoplay';
+ video.src = findMediaFile('video', 'content/test');
+ video.muted = true;
+ video.loop = true;
+ video.style.position = 'absolute';
+ video.style.top = '-10000px';
+ document.body.appendChild(video);
+ }
+
+ var inscreen = document.querySelector('#inscreen');
+ var offscreen = document.querySelector('#offscreen');
+ var offscreenNoAutoplay = document.querySelector('#offscreen-no-autoplay');
+ var audio = document.querySelector('audio');
+
+ inscreen.addEventListener('playing', t.step_func(function() {
+ // The offscreen videos shouldn't play.
+ assert_true(offscreen.paused);
+ assert_true(offscreenNoAutoplay.paused);
+ assert_false(inscreen.paused);
+ assert_true(audio.paused, 'audio should not play');
+
+ // Both autoplay videos should have loaded.
+ assert_greater_than_equal(inscreen.readyState, HTMLMediaElement.HAVE_ENOUGH_DATA);
+ assert_greater_than_equal(offscreen.readyState, HTMLMediaElement.HAVE_ENOUGH_DATA);
+
+ // Move video in screen.
+ offscreen.style.top = '0px';
+
+ offscreen.addEventListener('playing', t.step_func(function() {
+ // Both autoplay videos should now play.
+ assert_false(inscreen.paused);
+ assert_false(offscreen.paused);
+ assert_true(offscreenNoAutoplay.paused);
+ assert_true(audio.paused, 'audio should not play');
+
+ // Call to play() should succeed regardless of visibility.
+ offscreenNoAutoplay.play();
+ offscreenNoAutoplay.addEventListener('playing', t.step_func_done(function() {
+ assert_false(inscreen.paused);
+ assert_false(offscreen.paused);
+ assert_false(offscreenNoAutoplay.paused);
+ assert_true(audio.paused, 'audio should not play');
+ }));
+ }));
+ }));
+ });
+
+</script>
+</body>
+</html>
esprehn 2016/06/21 21:28:41 no need for these

Powered by Google App Engine
This is Rietveld 408576698