Title: [152184] trunk/LayoutTests
Revision
152184
Author
[email protected]
Date
2013-06-28 12:55:18 -0700 (Fri, 28 Jun 2013)

Log Message

media/video-currentTime.html flakey
https://bugs.webkit.org/show_bug.cgi?id=118198

Reviewed by Jessie Berlin.

Update video-currentTime.html so that it listens for timeupdate events, and ends the test
after the first one with a >0 currentTime. Add a 3s watchdog for the failure case.

* media/video-currentTime-expected.txt:
* media/video-currentTime.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (152183 => 152184)


--- trunk/LayoutTests/ChangeLog	2013-06-28 19:09:52 UTC (rev 152183)
+++ trunk/LayoutTests/ChangeLog	2013-06-28 19:55:18 UTC (rev 152184)
@@ -1,3 +1,16 @@
+2013-06-28  Jer Noble  <[email protected]>
+
+        media/video-currentTime.html flakey
+        https://bugs.webkit.org/show_bug.cgi?id=118198
+
+        Reviewed by Jessie Berlin.
+
+        Update video-currentTime.html so that it listens for timeupdate events, and ends the test
+        after the first one with a >0 currentTime. Add a 3s watchdog for the failure case.
+
+        * media/video-currentTime-expected.txt:
+        * media/video-currentTime.html:
+
 2013-06-28  Bem Jones-Bey  <[email protected]>
 
         Update TestExpectations so new Shapes tests are skipped where the old ones were

Modified: trunk/LayoutTests/media/video-currentTime-expected.txt (152183 => 152184)


--- trunk/LayoutTests/media/video-currentTime-expected.txt	2013-06-28 19:09:52 UTC (rev 152183)
+++ trunk/LayoutTests/media/video-currentTime-expected.txt	2013-06-28 19:55:18 UTC (rev 152184)
@@ -2,6 +2,6 @@
 EVENT(canplaythrough)
 EXPECTED (video.currentTime == '0') OK
 EVENT(play)
-EXPECTED (video.currentTime > '0') OK
+video.currentTime > 0 OK
 END OF TEST
 

Modified: trunk/LayoutTests/media/video-currentTime.html (152183 => 152184)


--- trunk/LayoutTests/media/video-currentTime.html	2013-06-28 19:09:52 UTC (rev 152183)
+++ trunk/LayoutTests/media/video-currentTime.html	2013-06-28 19:55:18 UTC (rev 152184)
@@ -4,12 +4,14 @@
 <script>
     testExpected("video.currentTime", 0)
     waitForEvent('canplaythrough', function() { testExpected("video.currentTime", 0); } );
-    video.addEventListener('canplaythrough', function() { video.play(); setTimeout(testCurrentTime, 500) });
+    video.addEventListener('canplaythrough', function() { video.play(); });
     waitForEvent('play');
-    function testCurrentTime()
-    {
-        testExpected("video.currentTime", 0, '>')
-        endTest();
-    }
+    video.addEventListener('timeupdate', function() {
+        if (video.currentTime > 0) {
+            logResult(true, 'video.currentTime > 0');
+            endTest();
+        }
+    });
     video.src = "" "content/test");
+    failTestIn(3000);
 </script>
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to