summaryrefslogtreecommitdiffstats
path: root/content/browser/loader/sync_resource_handler.h
Commit message (Collapse)AuthorAgeFilesLines
* Standardize usage of virtual/override/final in content/browser/dcheng2014-10-211-17/+16
| | | | | | | | | | | | This patch was automatically generated by applying clang fixit hints generated by the plugin to the source tree. BUG=417463 TBR=sky@chromium.org Review URL: https://codereview.chromium.org/667943003 Cr-Commit-Position: refs/heads/master@{#300469}
* Replace OVERRIDE and FINAL with override and final in content/browser/loaderanand.ratn2014-10-071-9/+9
| | | | | | | | BUG=417463 Review URL: https://codereview.chromium.org/635713002 Cr-Commit-Position: refs/heads/master@{#298489}
* Plumb redirect info out of net, through content, and into child processes.davidben@chromium.org2014-08-151-1/+1
| | | | | | | | | | | | | This saves the logic in PluginURLFetcher and WebURLLoader that has to replicate the method-munging, referrer-munging, and first-party-URL-munging logic that is already done in net/. BUG=384609 Review URL: https://codereview.chromium.org/398903002 Cr-Commit-Position: refs/heads/master@{#290005} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@290005 0039d316-1c4b-4281-b951-d872f2087c98
* Remove request_id parameter from ResourceHandler methods.davidben@chromium.org2014-05-281-21/+9
| | | | | | | | | | | | AsyncResourceHandler and downloads are the only components which consume it. Everyone else just passes it down. Pull it out of ResourceRequestInfo where needed. Other components need not deal with it. BUG=376003 Review URL: https://codereview.chromium.org/298253006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@273368 0039d316-1c4b-4281-b951-d872f2087c98
* Add transfer size paramater to didFinishLoading [2/3]eustas@chromium.org2014-02-101-0/+1
| | | | | | | | | | | | | | | | | | | Mini DD: https://docs.google.com/a/chromium.org/document/d/1kAKnBKofLc3Tex_JPveubwJcQLT3BhIL3xNQyjvRIps/edit?usp=sharing Most of files contain only signature changes. There are 3 non-trivial changes: 1) ResourceMsg_RequestComplete IPC message receives struct instead of distinct parameters. 2) Use URLRequest::GetTotalReceivedBytes instead of DevToolsNetLogObserver::GetAndResetEncodedDataLength 3) Remove unused code in DevToolsNetLogObserver. BUG=111052 Review URL: https://codereview.chromium.org/143263002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250069 0039d316-1c4b-4281-b951-d872f2087c98
* Adds the new URLRequest::OnBeforeNetworkStart hook to thejkarlin@chromium.org2014-01-091-0/+3
| | | | | | | | | | | | | | | | ResourceThrottle so that the ResourceScheduler can eventually throttle closer to the start of network usage. This is the second step in the design doc: https://docs.google.com/document/d/1TSI3jwozVB_nueWJxzi8uKbgDfsGZRZqw8tvtD-P1Iw/edit?usp=sharing The third step is to create the new ResourceScheduler. BUG=328741 Review URL: https://codereview.chromium.org/129173002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@243994 0039d316-1c4b-4281-b951-d872f2087c98
* Make ResourceHandler::OnResponseCompleted take a defer output parameter.davidben@chromium.org2013-11-181-2/+3
| | | | | | | | | | | | | | | Every other method of ResourceHandler returns false to abort and sets *defer to cancel. Between the StreamResourceHandler fix here and https://src.chromium.org/viewvc/chrome?view=rev&revision=233661, there were twice as many ResourceHandlers that mistakenly deferred shutdown as intentionally. (RedirectToFileResourceHandler and CrossSiteResourceHandler do so intentionally.) Make this consistent so it's not as confusing. BUG=none Review URL: https://codereview.chromium.org/74373002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@235631 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up ResourceHandler API.creis@chromium.org2013-10-151-2/+1
| | | | | | | | | | | | | | | 1. Move URLRequest member to ResourceHandler base class and add convenience functions for subclasses. 2. Make OnWillRead take a scoped_refptr to avoid leaking the buffer. (Attempt 2, after being reverted for possibly unrelated crashes.) BUG=295239 TEST=No behavior change. TBR=darin Review URL: https://codereview.chromium.org/26420005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@228741 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 227318 "Clean up ResourceHandler API."creis@chromium.org2013-10-081-1/+2
| | | | | | | | | | | | | | | | | | | | Could be causing http://crbug.com/305333. > Clean up ResourceHandler API. > > 1. Move URLRequest member to ResourceHandler base class and add convenience functions for subclasses. > 2. Make OnWillRead take a scoped_refptr to avoid leaking the buffer. > > BUG=295239 > TEST=No behavior change. > > Review URL: https://codereview.chromium.org/25536005 TBR=creis@chromium.org Review URL: https://codereview.chromium.org/26472004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227594 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up ResourceHandler API.creis@chromium.org2013-10-071-2/+1
| | | | | | | | | | | | 1. Move URLRequest member to ResourceHandler base class and add convenience functions for subclasses. 2. Make OnWillRead take a scoped_refptr to avoid leaking the buffer. BUG=295239 TEST=No behavior change. Review URL: https://codereview.chromium.org/25536005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227318 0039d316-1c4b-4281-b951-d872f2087c98
* Make ResourceHandlers stateless with respect to child/routing/request IDs.creis@chromium.org2013-09-251-5/+1
| | | | | | | | | | | | This lets us transfer an existing handler chain to a new process. BUG=238331 TEST=Follow a link that redirects to the Chrome Web Store. R=ajwong@chromium.org, darin@chromium.org, mpcomplete@chromium.org Review URL: https://codereview.chromium.org/23180005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225263 0039d316-1c4b-4281-b951-d872f2087c98
* Modify ResourceMessageFilter so that it can support attaching to ↵jam@chromium.org2013-09-091-0/+3
| | | | | | | | | | | | | PluginProcessHost. Previously, a ResourceMessageFilter could assume that its client has one ResourceContext. For plugins this assumption won't hold anymore, as one plugin process serves many renderers. BUG=286074 R=ananta@chromium.org Review URL: https://codereview.chromium.org/23851010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222077 0039d316-1c4b-4281-b951-d872f2087c98
* Minor cleanup: make a method pure virtualmichaeln@chromium.org2012-12-201-0/+1
| | | | | | | | BUG=98443 Review URL: https://chromiumcodereview.appspot.com/11645004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@174110 0039d316-1c4b-4281-b951-d872f2087c98
* Add content/browser/loader/ for resource loading related classes.darin@chromium.org2012-12-051-0/+74
This infrastructure is shared by both renderer and worker processes, so having it live in renderer_host/ was not quite right. This change also nicely reduces the number of files in renderer_host/. R=jam@chromium.org Review URL: https://codereview.chromium.org/11414299 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171194 0039d316-1c4b-4281-b951-d872f2087c98