diff options
| author | creis@chromium.org <creis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-07 19:45:52 +0000 |
|---|---|---|
| committer | creis@chromium.org <creis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-07 19:45:52 +0000 |
| commit | 66ab87e8ebc518ab54238f9899f813311844997b (patch) | |
| tree | b26bbd221e2668c33d84ab9ca92cc45a01828080 /content/browser/loader/async_resource_handler.h | |
| parent | c745d405c8ac5a39c4ede5ccafe3445878b3767b (diff) | |
| download | chromium_src-66ab87e8ebc518ab54238f9899f813311844997b.zip chromium_src-66ab87e8ebc518ab54238f9899f813311844997b.tar.gz chromium_src-66ab87e8ebc518ab54238f9899f813311844997b.tar.bz2 | |
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
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227318 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/loader/async_resource_handler.h')
| -rw-r--r-- | content/browser/loader/async_resource_handler.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/content/browser/loader/async_resource_handler.h b/content/browser/loader/async_resource_handler.h index 0992841..4d980d2 100644 --- a/content/browser/loader/async_resource_handler.h +++ b/content/browser/loader/async_resource_handler.h @@ -50,7 +50,7 @@ class AsyncResourceHandler : public ResourceHandler, const GURL& url, bool* defer) OVERRIDE; virtual bool OnWillRead(int request_id, - net::IOBuffer** buf, + scoped_refptr<net::IOBuffer>* buf, int* buf_size, int min_size) OVERRIDE; virtual bool OnReadCompleted(int request_id, @@ -73,7 +73,6 @@ class AsyncResourceHandler : public ResourceHandler, void ResumeIfDeferred(); scoped_refptr<ResourceBuffer> buffer_; - net::URLRequest* request_; ResourceDispatcherHostImpl* rdh_; // Number of messages we've sent to the renderer that we haven't gotten an |