Test file errors in downloads.

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/logging.h"
@@ skipping 232 matching lines @@
   if (vector_size > kLoadsToWrite)
     StartPauseTimer();
 
   return true;
 }
 
 bool DownloadResourceHandler::OnResponseCompleted(
     int request_id,
     const net::URLRequestStatus& status,
     const std::string& security_info) {
+  VLOG(20) << __FUNCTION__ << "()" << DebugString()
+           << " request_id = " << request_id
+           << " status.status() = " << status.status()
+           << " status.error() = " << status.error();
   if (download_id_.IsValid()) {
     OnResponseCompletedInternal(request_id, status, security_info);
   } else {
     // We got cancelled before the task which sets the id ran on the IO thread.
     // Wait for it.
     BrowserThread::PostTaskAndReply(
         BrowserThread::UI, FROM_HERE,
         base::Bind(&base::DoNothing),
         base::Bind(&DownloadResourceHandler::OnResponseCompletedInternal, this,
                    request_id, status, security_info));
   }
   return true;
 }
 
 void DownloadResourceHandler::OnResponseCompletedInternal(
     int request_id,
     const net::URLRequestStatus& status,
     const std::string& security_info) {
-  VLOG(20) << __FUNCTION__ << "()" << DebugString()
+  // NOTE: |request_| may be a dangling pointer at this point.
+  VLOG(20) << __FUNCTION__ << "()"
            << " request_id = " << request_id
            << " status.status() = " << status.status()
            << " status.error() = " << status.error();
   net::Error error_code = net::OK;
   if (status.status() == net::URLRequestStatus::FAILED)
     error_code = static_cast<net::Error>(status.error());  // Normal case.
   // ERR_CONNECTION_CLOSED is allowed since a number of servers in the wild
   // advertise a larger Content-Length than the amount of bytes in the message
   // body, and then close the connection. Other browsers - IE8, Firefox 4.0.1,
   // and Safari 5.0.4 - treat the download as complete in this case, so we
@@ skipping 142 matching lines @@
                             " }",
                             request_ ?
                                 request_->url().spec().c_str() :
                                 "<NULL request>",
                             download_id_.local(),
                             global_id_.child_id,
                             global_id_.request_id,
                             render_view_id_,
                             save_info_.file_path.value().c_str());
 }