[Webkit-unassigned] [Bug 90272] <video> element delays document load event for ~ 3 seconds

[bugzilla-daemon at webkit.org]

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


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

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #204573|review?                     |review-
               Flag|                            |




--- Comment #6 from Eric Carlson <eric.carlson at apple.com>  2013-06-13 08:13:30 PST ---
(From update of attachment 204573)
View in context: https://bugs.webkit.org/attachment.cgi?id=204573&action=review

> LayoutTests/media/media-preload-no-delay-loadevent.html:8
> +function finish(){

Nit: a function's opening brace should be on a new line.

> LayoutTests/media/media-preload-no-delay-loadevent.html:12
> +		}

Eek, a tab!

> LayoutTests/media/media-preload-no-delay-loadevent.html:22
> +		document.getElementById('load-time').innerHTML = loadTime + '  PASS';

Logging the time it takes to load will make this test flakey because not all bots run at the same speed.

> LayoutTests/media/media-preload-no-delay-loadevent.html:30
> +	<source src="void.mp4">

This test should use a valid media file that can be loaded by the platform being tested. See how other tests use "findMediaFile" to choose a platform appropriate test file.

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