Bug 35793 - media/video-preload.html fails
Summary: media/video-preload.html fails
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Linux
: P2 Normal
Assignee: Nobody
: 30378 36068 (view as bug list)
Depends on:
Reported: 2010-03-05 09:27 PST by Eric Carlson
Modified: 2010-03-15 08:20 PDT (History)
4 users (show)

See Also:

respect preload attribute when loading (5.05 KB, patch)
2010-03-12 11:55 PST, Gustavo Noronha (kov)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Carlson 2010-03-05 09:27:24 PST
The new 'preload' functional test fails, "preload=none" loads enough to fire the 'loadedmetadata' event. I don't know enough about how your media engine works to attempt a fix.
Comment 1 Eric Carlson 2010-03-05 09:33:29 PST
Added to skip list in r55588.
Comment 2 Gustavo Noronha (kov) 2010-03-12 11:52:30 PST
*** Bug 36068 has been marked as a duplicate of this bug. ***
Comment 3 Gustavo Noronha (kov) 2010-03-12 11:55:20 PST
Created attachment 50618 [details]
respect preload attribute when loading
Comment 4 Gustavo Noronha (kov) 2010-03-12 12:11:50 PST
Landed as r55927.
Comment 5 Dirk Schulze 2010-03-12 22:57:25 PST
(In reply to comment #4)
> Landed as r55927.

This broke the bots. You call url without giving it to commitLoad().
Comment 6 Dirk Schulze 2010-03-13 05:44:54 PST
Rolled the patch out in r55962.
Comment 7 Philippe Normand 2010-03-15 03:17:31 PDT
*** Bug 30378 has been marked as a duplicate of this bug. ***
Comment 8 Gustavo Noronha (kov) 2010-03-15 08:11:55 PDT
(In reply to comment #6)
> Rolled the patch out in r55962.

Thanks for catching it, and cleaning up the mess! I only waited for the release bot. That'll teach me =D.
Comment 9 Gustavo Noronha (kov) 2010-03-15 08:20:33 PDT
Re-landed with the build fixage as r55995.