[Downloads] Do not store error responses during resumption.

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 111 matching lines @@
     BrowserThread::PostTask(
         BrowserThread::UI, FROM_HERE,
         base::Bind(&DeleteOnUIThread, base::Passed(&tab_info_)));
   }
 }
 
 bool DownloadResourceHandler::OnRequestRedirected(
     const net::RedirectInfo& redirect_info,
     ResourceResponse* response,
     bool* defer) {
-  return true;
+  DownloadInterruptReason result = core_.OnRequestRedirected(redirect_info);
+  if (result == DOWNLOAD_INTERRUPT_REASON_NONE)
+    return true;
+
+  // Allow the interrupt reason to propagate up the stack before we receive the
+  // OnResponseCompleted() call. The ResourceLoader will treat this as an ABORT,
+  // hence OnResponseCompleted() may conclude a more generic interrupt reason.
+  // We want the specific interrupt reason to be dispatched instead.
+  CallStartedCB(result);
+  return false;
 }
 
 // Send the download creation information to the download thread.
 bool DownloadResourceHandler::OnResponseStarted(
     ResourceResponse* response,
     bool* defer) {
   scoped_ptr<DownloadCreateInfo> create_info;
   scoped_ptr<ByteStreamReader> stream_reader;
 
-  core_.OnResponseStarted(&create_info, &stream_reader);
+  DownloadInterruptReason result =
+      core_.OnResponseStarted(&create_info, &stream_reader);
+  if (result != DOWNLOAD_INTERRUPT_REASON_NONE) {
+    CallStartedCB(result);
+    return false;
+  }
 
   const ResourceRequestInfoImpl* request_info = GetRequestInfo();
   create_info->download_id = download_id_;
   create_info->has_user_gesture = request_info->HasUserGesture();
   create_info->transition_type = request_info->GetPageTransition();
   create_info->request_handle.reset(new DownloadRequestHandle(
       AsWeakPtr(), request_info->GetChildID(), request_info->GetRouteID(),
       request_info->GetRequestID(), request_info->frame_tree_node_id()));
 
   // The MIME type in ResourceResponse is the product of
@@ skipping 87 matching lines @@
                             " }",
                             request() ?
                                 request()->url().spec().c_str() :
                                 "<NULL request>",
                             info->GetChildID(),
                             info->GetRequestID(),
                             info->GetRouteID());
 }
 
 }  // namespace content