Substitute ResourceHandlers with dummy ResourceHandler while request is transferred

content/browser/renderer_host/paused_resource_handler.h

+// Copyright (c) 2011 The Chromium Authors. All rights reserved.

[Charlie Reis, 2012/01/06 17:54:52]

New files should use 2012, right?

[battre, 2012/01/06 18:39:04]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CONTENT_BROWSER_RENDERER_HOST_PAUSED_RESOURCE_HANDLER_H_
+#define CONTENT_BROWSER_RENDERER_HOST_PAUSED_RESOURCE_HANDLER_H_
+#pragma once
+
+#include "base/memory/ref_counted.h"
+#include "content/browser/renderer_host/resource_handler.h"
+
+// ResourceHandler that DCHECKs on all events but canceling and failing of
+// requests. This is used for requests that are paused to be transferred to
+// a new renderer.
+class PausedResourceHandler : public ResourceHandler {

[Charlie Reis, 2012/01/06 17:54:52]

Sorry, I'm not sure I agree on the name.  There's a lot of cases that we pause
requests with the intention of resuming them.  This seems like it will be
confusing to people who might think those cases should use a
PausedResourceHandler.

Can we come up with a name that indicates this isn't expected to be unpaused? 
Something like TransferredResourceHandler?  That may not great either, though,
since it's close to TransferNavigationResourceHandler.

[battre, 2012/01/06 18:39:04]

Lacking a better proposal I have renamed the class to
TransferredResourceHandler.

Done.

+ public:
+  // As the PausedResourceHandler is constructed and subsituted from code
+  // of another ResourceHandler, we need to make sure that this other handler
+  // does not lose its last reference and gets destroyed by being substituted.
+  // Therefore, we retain a reference to |old_handlers| that prevents the
+  // destruction.
+  PausedResourceHandler(ResourceHandler* old_handlers);
+
+  // ResourceHandler implementation:
+  virtual bool OnUploadProgress(int request_id,
+                                uint64 position,
+                                uint64 size) OVERRIDE;
+  virtual bool OnRequestRedirected(int request_id,
+                                   const GURL& new_url,
+                                   content::ResourceResponse* response,
+                                   bool* defer) OVERRIDE;
+  virtual bool OnResponseStarted(int request_id,
+                                 content::ResourceResponse* response) OVERRIDE;
+  virtual bool OnWillStart(int request_id,
+                           const GURL& url,
+                           bool* defer) OVERRIDE;
+  virtual bool OnWillRead(int request_id,
+                          net::IOBuffer** buf,
+                          int* buf_size,
+                          int min_size) OVERRIDE;
+  virtual bool OnReadCompleted(int request_id,
+                               int* bytes_read) OVERRIDE;
+  virtual bool OnResponseCompleted(int request_id,
+                                   const net::URLRequestStatus& status,
+                                   const std::string& security_info) OVERRIDE;
+  virtual void OnRequestClosed() OVERRIDE;
+  virtual void OnDataDownloaded(int request_id,
+                                int bytes_downloaded) OVERRIDE;
+
+ private:
+  virtual ~PausedResourceHandler();
+
+  // Reference to the old ResourceHandlers that were active before this one
+  // replaced them. We need to retain a reference here because we replace
+  // the old handlers while they are being executed. Replacing them without
+  // retaining the reference might delete the last reference and destroy them.
+  scoped_refptr<ResourceHandler> old_handlers_;
+
+  DISALLOW_COPY_AND_ASSIGN(PausedResourceHandler);
+};
+
+#endif  // CONTENT_BROWSER_RENDERER_HOST_PAUSED_RESOURCE_HANDLER_H_