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

See Also:

Attachments
respect preload attribute when loading (5.05 KB, patch)
2010-03-12 11:55 PST, Gustavo Noronha (kov) 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

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.