Changeset 79401 in webkit


Ignore:
Timestamp:
Feb 22, 2011 9:51:14 PM (13 years ago)
Author:
commit-queue@webkit.org
Message:

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:
Location:
trunk/LayoutTests
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r79400 r79401  
     12011-02-22  Anna Cavender  <annacc@chromium.org>
     2
     3        Reviewed by Eric Carlson.
     4
     5        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
     6        https://bugs.webkit.org/show_bug.cgi?id=54910
     7
     8        * media/video-replaces-poster.html:
     9
    1102011-02-22  Ryosuke Niwa  <rniwa@webkit.org>
    211

  • trunk/LayoutTests/media/video-replaces-poster.html

    r78778 r79401  
    22<html>
    33<head>
    4     <script src="video-paint-test.js"></script>
    54    <script src="../http/tests/media/media-file.js"></script>
    65    <script type="text/javascript" charset="utf-8">
     6        if (window.layoutTestController)
     7            layoutTestController.waitUntilDone();
    78        function doSetup()
    89        {
    … …  
    1314                video.currentTime = 1;  // so the snapshot always has the same frame.
    1415            });
    15             init();
     16            video.addEventListener("seeked", function() {
     17                if (window.layoutTestController)
     18                    layoutTestController.notifyDone();
     19            });
    1620        }
    1721        window.addEventListener('load', doSetup, false);
Note: See TracChangeset for help on using the changeset viewer.