Title: [111029] trunk/LayoutTests
Revision
111029
Author
[email protected]
Date
2012-03-16 11:21:16 -0700 (Fri, 16 Mar 2012)

Log Message

[chromium] compositing tests with videos fail in chromium DumpRenderTree, seeking doesn't appear to work
https://bugs.webkit.org/show_bug.cgi?id=76511

Patch by Aaron Colwell <[email protected]> on 2012-03-16
Reviewed by Eric Carlson.

* compositing/resources/media-testing.js:
(ensureFrameVisible.playingHandler):
(ensureFrameVisible.seekedHandler):
(ensureFrameVisible):

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (111028 => 111029)


--- trunk/LayoutTests/ChangeLog	2012-03-16 18:12:14 UTC (rev 111028)
+++ trunk/LayoutTests/ChangeLog	2012-03-16 18:21:16 UTC (rev 111029)
@@ -1,3 +1,15 @@
+2012-03-16  Aaron Colwell  <[email protected]>
+
+        [chromium] compositing tests with videos fail in chromium DumpRenderTree, seeking doesn't appear to work
+        https://bugs.webkit.org/show_bug.cgi?id=76511
+
+        Reviewed by Eric Carlson.
+
+        * compositing/resources/media-testing.js:
+        (ensureFrameVisible.playingHandler):
+        (ensureFrameVisible.seekedHandler):
+        (ensureFrameVisible):
+
 2012-03-08  Jer Noble  <[email protected]>
 
         Support W3C Full Screen API proposal

Modified: trunk/LayoutTests/compositing/resources/media-testing.js (111028 => 111029)


--- trunk/LayoutTests/compositing/resources/media-testing.js	2012-03-16 18:12:14 UTC (rev 111028)
+++ trunk/LayoutTests/compositing/resources/media-testing.js	2012-03-16 18:21:16 UTC (rev 111029)
@@ -4,13 +4,18 @@
 
 function ensureFrameVisible(video, videoRenderedCallback)
 {
-    video.addEventListener('timeupdate', function() {
+    var playingHandler = function() {
         video.pause();
         video.currentTime = 0;
+    };
+
+    var seekedHandler = function() {
         if (videoRenderedCallback)
             videoRenderedCallback();
-    }, false);
-    
+    };
+
+    video.addEventListener('playing', playingHandler, false);
+    video.addEventListener('seeked', seekedHandler, false);
     video.play();
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to