Make ResourceHandler be non-refcounted. Eliminate the OnRequestClosed

method in favor of ~ResourceHandler.

This required making a couple ResourceHandlers support weak pointers.

The only non-trivial changes are related to DownloadResourceHandler.
The StartOnUIThread method becomes a static StartDownloadOnUIThread
method, and the call to SetDownloadID and CallStartedCB are now hidden
behind a Callback<void(DownloadId)>, which gets posted to the IO thread.

R=rdsmith@chromium.org,jam@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10455009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139874 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 1 insertions, 3 deletions

diff --git a/content/browser/renderer_host/sync_resource_handler.h b/content/browser/renderer_host/sync_resource_handler.h
index c2c1446..f07e7e1 100644
--- a/content/browser/renderer_host/sync_resource_handler.h
+++ b/content/browser/renderer_host/sync_resource_handler.h
@@ -32,6 +32,7 @@ class SyncResourceHandler : public ResourceHandler {
                       const GURL& url,
                       IPC::Message* result_message,
                       ResourceDispatcherHostImpl* resource_dispatcher_host);
+  virtual ~SyncResourceHandler();
 
   virtual bool OnUploadProgress(int request_id,
                                 uint64 position,
@@ -56,13 +57,10 @@ class SyncResourceHandler : public ResourceHandler {
   virtual bool OnResponseCompleted(int request_id,
                                    const net::URLRequestStatus& status,
                                    const std::string& security_info) OVERRIDE;
-  virtual void OnRequestClosed() OVERRIDE;
 
  private:
   enum { kReadBufSize = 3840 };
 
-  virtual ~SyncResourceHandler();
-
   scoped_refptr<net::IOBuffer> read_buffer_;
 
   SyncLoadResult result_;