[Webkit-unassigned] [Bug 54910] media/video-replaces-poster.html fails on all platforms

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Mar 1 09:51:23 PST 2011


https://bugs.webkit.org/show_bug.cgi?id=54910


Eric Carlson <eric.carlson at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #84132|review?                     |review+
               Flag|                            |




--- Comment #27 from Eric Carlson <eric.carlson at apple.com>  2011-03-01 09:51:23 PST ---
(From update of attachment 84132)
View in context: https://bugs.webkit.org/attachment.cgi?id=84132&action=review

r+ but I would prefer to have at least the comment updated before landing.

> LayoutTests/media/video-replaces-poster.html:25
> +        function checkExpected(actual, expected)
> +        {
> +            if (actual != expected)
> +                document.writeln("<p>FAILURE: expected " + expected + ", actual " + actual + "</p>");
> +        }

As long as this test needs to be updated, can you include video-test.js and use one of its functions?

> LayoutTests/media/video-replaces-poster.html:31
>      You should see the video play below.</p>

This comment is no longer correct.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list