Refactor ResourceHandler API.

content/browser/loader/layered_resource_handler.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/loader/layered_resource_handler.h"
 
 #include <utility>
 
 #include "base/logging.h"
+#include "content/public/browser/resource_controller.h"
 
 namespace content {
 
 LayeredResourceHandler::LayeredResourceHandler(
     net::URLRequest* request,
     std::unique_ptr<ResourceHandler> next_handler)
     : ResourceHandler(request), next_handler_(std::move(next_handler)) {}
 
 LayeredResourceHandler::~LayeredResourceHandler() {
 }
 
-void LayeredResourceHandler::SetController(ResourceController* controller) {
-  ResourceHandler::SetController(controller);
-
-  // Pass the controller down to the next handler.  This method is intended to
-  // be overriden by subclasses of LayeredResourceHandler that need to insert a
-  // different ResourceController.
-
+void LayeredResourceHandler::OnRequestRedirected(
+    const net::RedirectInfo& redirect_info,
+    ResourceResponse* response,
+    std::unique_ptr<ResourceController> controller) {
   DCHECK(next_handler_.get());
-  next_handler_->SetController(controller);
+  next_handler_->OnRequestRedirected(redirect_info, response,
+                                     std::move(controller));
 }
 
-bool LayeredResourceHandler::OnRequestRedirected(
-    const net::RedirectInfo& redirect_info,
+void LayeredResourceHandler::OnResponseStarted(
     ResourceResponse* response,
-    bool* defer) {
+    std::unique_ptr<ResourceController> controller) {
   DCHECK(next_handler_.get());
-  return next_handler_->OnRequestRedirected(redirect_info, response, defer);
+  next_handler_->OnResponseStarted(response, std::move(controller));
 }
 
-bool LayeredResourceHandler::OnResponseStarted(ResourceResponse* response,
-                                               bool* defer) {
+void LayeredResourceHandler::OnWillStart(
+    const GURL& url,
+    std::unique_ptr<ResourceController> controller) {
   DCHECK(next_handler_.get());
-  return next_handler_->OnResponseStarted(response, defer);
-}
-
-bool LayeredResourceHandler::OnWillStart(const GURL& url,
-                                         bool* defer) {
-  DCHECK(next_handler_.get());
-  return next_handler_->OnWillStart(url, defer);
+  next_handler_->OnWillStart(url, std::move(controller));
 }
 
 bool LayeredResourceHandler::OnWillRead(scoped_refptr<net::IOBuffer>* buf,
                                         int* buf_size,
                                         int min_size) {
   DCHECK(next_handler_.get());
   return next_handler_->OnWillRead(buf, buf_size, min_size);
 }
 
-bool LayeredResourceHandler::OnReadCompleted(int bytes_read, bool* defer) {
+void LayeredResourceHandler::OnReadCompleted(
+    int bytes_read,
+    std::unique_ptr<ResourceController> controller) {
   DCHECK(next_handler_.get());
-  return next_handler_->OnReadCompleted(bytes_read, defer);
+  next_handler_->OnReadCompleted(bytes_read, std::move(controller));
 }
 
 void LayeredResourceHandler::OnResponseCompleted(
     const net::URLRequestStatus& status,
-    bool* defer) {
+    std::unique_ptr<ResourceController> controller) {
   DCHECK(next_handler_.get());
-  next_handler_->OnResponseCompleted(status, defer);
+  next_handler_->OnResponseCompleted(status, std::move(controller));
 }
 
 void LayeredResourceHandler::OnDataDownloaded(int bytes_downloaded) {
   DCHECK(next_handler_.get());
   next_handler_->OnDataDownloaded(bytes_downloaded);
 }
 
 }  // namespace content