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

Unified Diff: LayoutTests/media/video-currentTime-before-have-metadata.html

Issue 517593003: Allow HTMLMediaElement.currentTime to be set before the transition to HAVE_METADATA (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Fast-Forward to origin/master and rebase Created 6 years, 3 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-currentTime-before-have-metadata.html
diff --git a/LayoutTests/media/video-currentTime-before-have-metadata.html b/LayoutTests/media/video-currentTime-before-have-metadata.html
new file mode 100644
index 0000000000000000000000000000000000000000..d8c68c8d72fc2e98a87e14a734b91a20447dcf5f
--- /dev/null
+++ b/LayoutTests/media/video-currentTime-before-have-metadata.html
@@ -0,0 +1,34 @@
+<!DOCTYPE html>
+<html>
+ <head>
+ <script src=media-file.js></script>
+ <script src=video-test.js></script>
+ <script>
+ function onWindowLoad(e)
+ {
+ video = document.getElementById('video');
+
+ video.src = findMediaFile("video", "content/test");
+ testExpected("video.currentTime", 0);
+ video.currentTime = 1;
philipj_slow 2014/09/10 09:57:05 Add testExpected("video.currentTime", 1) after thi
amogh.bihani 2014/09/10 11:18:33 Done.
+
+ waitForEvent('loadedmetadata', function()
+ {
+ testExpected("video.currentTime", 1);
+ });
+
+ waitForEvent('seeked', function()
philipj_slow 2014/09/10 09:57:05 This can be simplified to waitForEventAndEnd("seek
amogh.bihani 2014/09/10 11:18:33 Done.
+ {
+ endTest();
+ });
+ }
+
+ window.addEventListener('load', onWindowLoad, false);
+ </script>
+ </head>
+ <body>
+ <video controls id="video"></video>
+ <p>Test currentTime values when setting while HAVE_NOTHING.</p>
+ <br/>
+ </body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698