summaryrefslogtreecommitdiffstats
path: root/webkit/media/active_loader.cc
Commit message (Collapse)AuthorAgeFilesLines
* Move webkit/media/ into webkit/renderer/mediajamesr@chromium.org2013-05-301-26/+0
| | | | | | | | | | | | This code is only used in renderer processes, this just moves the files to reflect that fact in the directory structure so we can verify it with DEPS. BUG=237267 Review URL: https://chromiumcodereview.appspot.com/15842010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@203248 0039d316-1c4b-4281-b951-d872f2087c98
* Update refernces to Blink's Platform API (webkit)abarth@chromium.org2013-05-301-1/+1
| | | | | | | | | | | | | | These headers have moved from Source/Platform/chromium/public to public/platform. This CL updates src/webkit's references to the old location to point to the new location. After this CL lands, I'll remove the forwarding headers that are letting these references still work. TBR=jamesr@chromium.org BUG=239545 Review URL: https://chromiumcodereview.appspot.com/15937016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@203020 0039d316-1c4b-4281-b951-d872f2087c98
* Update some #includes in webkit/media for headers in the new Platform directorypilgrim@chromium.org2013-01-151-1/+1
| | | | | | Review URL: https://chromiumcodereview.appspot.com/11887029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176988 0039d316-1c4b-4281-b951-d872f2087c98
* Remove ActiveLoader::Cancel().scherkus@chromium.org2012-01-131-5/+1
| | | | | | | | | | | This will force clients to rely on the destructor to cancel loaders instead and prevents the situation where you hold onto an ActiveLoader that has been cancelled. BufferedResourceLoader did this inside of didFail(), which led to WillFulfillRead() returning true, which led to a hang since the loader had been cancelled. BUG=106751 TEST=http/tests/appacache/video.html Review URL: http://codereview.chromium.org/9112035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117712 0039d316-1c4b-4281-b951-d872f2087c98
* Take advantage of the new Pass() machinery on scoped_ptr{,_malloc}.fischman@chromium.org2012-01-101-3/+3
| | | | | | | | | | | | | | | | | | | | | Pass() was announced in https://groups.google.com/a/chromium.org/d/topic/chromium-dev/RTd7rNxHjqk/discussion This CL replaces comments about ownership transfer (in all files whose paths contain media/) with the explicit passing of the appropriate scoper. The exceptions that are not touched by this CL: - scoped_refptr<> doesn't support Pass() and so is untouched. - media/audio code defines its own callback machinery, mimicking the old-style callbacks (pass by pointer, explicit deletes). I think that whole pile needs to be replaced with new-style (Bind) callbacks, so left it alone for now. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/9015015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117009 0039d316-1c4b-4281-b951-d872f2087c98
* Remove reference counting from BufferedResourceLoader.scherkus@chromium.org2011-12-141-10/+3
| | | | | | | | Reference counting was originally added in r23274 when we were still using ResourceLoaderBridge for URL loading. After switching to WebURLLoader in r69429 it turns out we don't need reference counting anymore. Review URL: http://codereview.chromium.org/8775062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114321 0039d316-1c4b-4281-b951-d872f2087c98
* Update these includes to use the new header locationsabarth@chromium.org2011-12-031-1/+1
| | | | | | | TBR=darin Review URL: http://codereview.chromium.org/8787003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112824 0039d316-1c4b-4281-b951-d872f2087c98
* Split a portion of BufferedResourceLoader into a separate class ActiveLoader.scherkus@chromium.org2011-12-021-0/+37
ActiveLoader encapsulates an active WebURLLoader and takes care of maintaining deferred status, references to parent object, and automatic cancelation during teardown. As a result of fixing the imbalanced reference counts to BufferedResourceLoader there were a few use-after-free bugs due to doing work after executing callbacks. The ordering has been updated to ensure that no more work is done after executing callbacks. BUG=100914 Review URL: http://codereview.chromium.org/8667002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112747 0039d316-1c4b-4281-b951-d872f2087c98