Refactor ResourceHandler API.

content/browser/download/download_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/download/download_resource_handler.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
@@ skipping 121 matching lines @@
 
 // static
 std::unique_ptr<ResourceHandler> DownloadResourceHandler::Create(
     net::URLRequest* request) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   std::unique_ptr<ResourceHandler> handler(
       new DownloadResourceHandler(request));
   return handler;
 }
 
-bool DownloadResourceHandler::OnRequestRedirected(
+void DownloadResourceHandler::OnRequestRedirected(
     const net::RedirectInfo& redirect_info,
     ResourceResponse* response,
-    bool* defer) {
-  return core_.OnRequestRedirected();
+    bool* defer_or_cancel) {
+  if (!core_.OnRequestRedirected()) {
+    *defer_or_cancel = true;
+    controller()->Cancel();
+  }
 }
 
 // Send the download creation information to the download thread.
-bool DownloadResourceHandler::OnResponseStarted(
-    ResourceResponse* response,
-    bool* defer) {
+void DownloadResourceHandler::OnResponseStarted(ResourceResponse* response,
+                                                bool* defer_or_cancel) {
   // The MIME type in ResourceResponse is the product of
   // MimeTypeResourceHandler.
-  return core_.OnResponseStarted(response->head.mime_type);
+  if (!core_.OnResponseStarted(response->head.mime_type)) {
+    *defer_or_cancel = true;
+    controller()->Cancel();
+  }
 }
 
-bool DownloadResourceHandler::OnWillStart(const GURL& url, bool* defer) {
-  return true;
-}
+void DownloadResourceHandler::OnWillStart(const GURL& url,
+                                          bool* defer_or_cancel) {}
 
 // Create a new buffer, which will be handed to the download thread for file
 // writing and deletion.
 bool DownloadResourceHandler::OnWillRead(scoped_refptr<net::IOBuffer>* buf,
                                          int* buf_size,
                                          int min_size) {
   return core_.OnWillRead(buf, buf_size, min_size);
 }
 
 // Pass the buffer to the download file writer.
-bool DownloadResourceHandler::OnReadCompleted(int bytes_read, bool* defer) {
-  return core_.OnReadCompleted(bytes_read, defer);
+void DownloadResourceHandler::OnReadCompleted(int bytes_read,
+                                              bool* defer_or_cancel) {
+  if (!core_.OnReadCompleted(bytes_read, defer_or_cancel)) {
+    *defer_or_cancel = true;
+    controller()->Cancel();
+  }
 }
 
 void DownloadResourceHandler::OnResponseCompleted(
     const net::URLRequestStatus& status,
     bool* defer) {
   core_.OnResponseCompleted(status);
 }
 
 void DownloadResourceHandler::OnDataDownloaded(int bytes_downloaded) {
   NOTREACHED();
@@ skipping 68 matching lines @@
                             " }",
                             request() ?
                                 request()->url().spec().c_str() :
                                 "<NULL request>",
                             info->GetChildID(),
                             info->GetRequestID(),
                             info->GetRouteID());
 }
 
 }  // namespace content