Bug 44157

Summary: [GStreamer] use ResourceHandle::setDefersLoading in WebKitWebSourceGStreamer
Product: WebKit Reporter: Philippe Normand <pnormand>
Component: MediaAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on: 44158    
Bug Blocks:    
Attachments:
Description Flags
proposed patch mrobinson: review+

Philippe Normand
Reported 2010-08-18 02:13:56 PDT
We should replace the platform-specific calls to pause/unpause the download with use of ResourceHandle::setDefersLoading
Attachments
proposed patch (2.50 KB, patch)
2010-08-18 03:22 PDT, Philippe Normand 		mrobinson: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Philippe Normand
Comment 1 2010-08-18 02:19:07 PDT
*** Bug 41512 has been marked as a duplicate of this bug. ***
Philippe Normand
Comment 2 2010-08-18 03:22:15 PDT
Created attachment 64682 [details] proposed patch
Martin Robinson
Comment 3 2010-08-23 08:40:33 PDT
Comment on attachment 64682 [details] proposed patch LGTM! Be sure to land after deferred loading lands in the soup backend.
Philippe Normand
Comment 4 2010-08-31 01:58:56 PDT
(In reply to comment #3) > (From update of attachment 64682 [details]) > LGTM! Be sure to land after deferred loading lands in the soup backend. Feel free to clear the r+ flag for now. Trying to land the patch from bug 44158 is counter-productive as it'd need work again after the SoupURILoader code lands from bug 44261
Philippe Normand
Comment 5 2010-09-02 00:11:09 PDT
Comment on attachment 64682 [details] proposed patch Clearing r+ flag as per comment 4
Martin Robinson
Comment 6 2010-11-23 08:49:17 PST
Comment on attachment 64682 [details] proposed patch Gogogogo!
Philippe Normand
Comment 7 2010-12-06 13:38:18 PST
Committed r73396: <http://trac.webkit.org/changeset/73396>
Note You need to log in before you can comment on or make changes to this bug.