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

Unified Diff: LayoutTests/media/video-currentTime-before-loadmetadata-media-fragment-uri.html

Issue 539103002: Seeking media fragment URI before loadeddata event (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@HAVE_NOTHING
Patch Set: Seeking media fragments if they have media controller 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-loadmetadata-media-fragment-uri.html
diff --git a/LayoutTests/media/video-currentTime-before-have-metadata.html b/LayoutTests/media/video-currentTime-before-loadmetadata-media-fragment-uri.html
similarity index 95%
copy from LayoutTests/media/video-currentTime-before-have-metadata.html
copy to LayoutTests/media/video-currentTime-before-loadmetadata-media-fragment-uri.html
index d8c68c8d72fc2e98a87e14a734b91a20447dcf5f..83c3de6e007a17f44ee338e57369944d14f74817 100644
--- a/LayoutTests/media/video-currentTime-before-have-metadata.html
+++ b/LayoutTests/media/video-currentTime-before-loadmetadata-media-fragment-uri.html
@@ -8,7 +8,7 @@
{
video = document.getElementById('video');
- video.src = findMediaFile("video", "content/test");
+ video.src = findMediaFile("video", "content/test") + "#t=2";
testExpected("video.currentTime", 0);
video.currentTime = 1;
@@ -28,7 +28,7 @@
</head>
<body>
<video controls id="video"></video>
- <p>Test currentTime values when setting while HAVE_NOTHING.</p>
+ <p>Test currentTime values when setting while HAVE_NOTHING for media fragment URI.</p>
philipj_slow 2014/09/10 12:25:48 Please name the two related tests similarly, maybe
amogh.bihani 2014/09/11 13:35:59 But the other file has nothing to do with before l
philipj_slow 2014/09/11 14:38:25 Sorry, I meant video-currentTime-before-have-metad
amogh.bihani 2014/09/12 11:02:23 Ah yes... this branch was made on top of that... A
<br/>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698