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

Side by Side Diff: third_party/WebKit/LayoutTests/http/tests/media/video-play-progress.html

Issue 1715303002: Add TODOs to convert from video-test.js to testharness.js (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add bug ref Created 4 years, 10 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 unified diff | Download patch
OLDNEW
1 <html> 1 <html>
2 <head> 2 <head>
3 <script src=../../media-resources/media-file.js></script> 3 <script src=../../media-resources/media-file.js></script>
4 <!-- TODO(philipj): Convert test to testharness.js. crbug.com/588956
5 (Please avoid writing new tests using video-test.js) -->
4 <script src=../../media-resources/video-test.js></script> 6 <script src=../../media-resources/video-test.js></script>
5 <script> 7 <script>
6 function start() { 8 function start() {
7 findMediaElement(); 9 findMediaElement();
8 10
9 var progressCount = 0; 11 var progressCount = 0;
10 12
11 // Make sure we've at least reached NETWORK_LOADING before waiting for progress. 13 // Make sure we've at least reached NETWORK_LOADING before waiting for progress.
12 waitForEvent('loadstart', function() { 14 waitForEvent('loadstart', function() {
13 waitForEventAndEnd('progress') 15 waitForEventAndEnd('progress')
14 } ); 16 } );
15 17
16 var mediaFile = findMediaFile("video", "resources/test"); 18 var mediaFile = findMediaFile("video", "resources/test");
17 video.src = "http://127.0.0.1:8000/" + mediaFile; 19 video.src = "http://127.0.0.1:8000/" + mediaFile;
18 run("video.play()"); 20 run("video.play()");
19 } 21 }
20 </script> 22 </script>
21 </head> 23 </head>
22 <body onload="start()"> 24 <body onload="start()">
23 <video></video> 25 <video></video>
24 <p>Test that at least one progress event is fired after starting to load t he video.</p> 26 <p>Test that at least one progress event is fired after starting to load t he video.</p>
25 </body> 27 </body>
26 </html> 28 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698