2011-02-22 Anna Cavender <annacc@chromium.org>
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Feb 2011 05:51:14 +0000 (05:51 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Feb 2011 05:51:14 +0000 (05:51 +0000)
        Reviewed by Eric Carlson.

        Fix for media/video-replaces-poster.html.  The test was failing because it assumes the results of a seek will be immediately available, when in fact the seek is asynchronous.  I did not remove the line in test-expectations because rebaselines for many (all?) platforms will likely be needed after this patch lands.  See also, http://code.google.com/p/chromium/issues/detail?id=60244
        https://bugs.webkit.org/show_bug.cgi?id=54910

        * media/video-replaces-poster.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@79401 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/media/video-replaces-poster.html

index 1594b6d..c780d6e 100644 (file)
@@ -1,3 +1,12 @@
+2011-02-22  Anna Cavender  <annacc@chromium.org>
+
+        Reviewed by Eric Carlson.
+
+        Fix for media/video-replaces-poster.html.  The test was failing because it assumes the results of a seek will be immediately available, when in fact the seek is asynchronous.  I did not remove the line in test-expectations because rebaselines for many (all?) platforms will likely be needed after this patch lands.  See also, http://code.google.com/p/chromium/issues/detail?id=60244
+        https://bugs.webkit.org/show_bug.cgi?id=54910
+
+        * media/video-replaces-poster.html:
+
 2011-02-22  Ryosuke Niwa  <rniwa@webkit.org>
 
         Unreviewed; second attempt to land Windows 7 expected failure results for new tests from r79318 
index f628ec1..2b49753 100644 (file)
@@ -1,9 +1,10 @@
 <!DOCTYPE html>
 <html>
 <head>
-    <script src="video-paint-test.js"></script>
     <script src="../http/tests/media/media-file.js"></script>
     <script type="text/javascript" charset="utf-8">
+        if (window.layoutTestController)
+            layoutTestController.waitUntilDone();
         function doSetup()
         {
             var video = document.getElementsByTagName('video')[0];
                 video.pause();
                 video.currentTime = 1;  // so the snapshot always has the same frame.
             });
-            init();
+            video.addEventListener("seeked", function() {
+                if (window.layoutTestController)
+                    layoutTestController.notifyDone();
+            });
         }
         window.addEventListener('load', doSetup, false);
     </script>