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

Side by Side Diff: LayoutTests/media/video-currentTime-set.html

Issue 17395006: Fix LayoutTests that assume canplay, playing, and canplaythrough will only fire once. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix event-attributes expectations. Created 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 <html> 1 <html>
2 <body> 2 <body>
3 3
4 <video controls></video> 4 <video controls></video>
5 5
6 <p>Test that setting currentTime changes the time, and that 'ended' event is fired in a reasonable amount of time</p> 6 <p>Test that setting currentTime changes the time, and that 'ended' event is fired in a reasonable amount of time</p>
7 7
8 <script src=media-file.js></script> 8 <script src=media-file.js></script>
9 <script src=video-test.js></script> 9 <script src=video-test.js></script>
10 <script> 10 <script>
11 waitForEvent('canplaythrough', 11 waitForEventOnce('canplaythrough',
12 function () { 12 function () {
13 waitForEventAndEnd('ended'); 13 waitForEventAndEnd('ended');
14 14
15 testExpected("video.currentTime", 0); 15 testExpected("video.currentTime", 0);
16 run("video.currentTime = video.duration - 0.2"); 16 run("video.currentTime = video.duration - 0.2");
17 }); 17 });
18 18
19 waitForEvent('seeked', 19 waitForEvent('seeked',
20 function () { 20 function () {
21 testExpected("video.currentTime.toFixed(2)", (video.duration - 0.2).toFi xed(2)); 21 testExpected("video.currentTime.toFixed(2)", (video.duration - 0.2).toFi xed(2));
22 run("video.play()"); 22 run("video.play()");
23 consoleWrite(""); 23 consoleWrite("");
24 }); 24 });
25 25
26 video.src = findMediaFile("video", "content/test"); 26 video.src = findMediaFile("video", "content/test");
27 </script> 27 </script>
28 28
29 </body> 29 </body>
30 </html> 30 </html>
OLDNEW
« no previous file with comments | « LayoutTests/media/video-currentTime-delay.html ('k') | LayoutTests/media/video-duration-known-after-eos.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698