Treat ERR_CONNECTION_CLOSED as end-of-data marker for downloads.

net/url_request/url_request_job.cc

Index: net/url_request/url_request_job.cc
diff --git a/net/url_request/url_request_job.cc b/net/url_request/url_request_job.cc
index 3073414d421426aa7e3b086300840e6069007db1..6235ed6f112c7d8406ca4ddb19faa2b56a1aedef 100644
--- a/net/url_request/url_request_job.cc
+++ b/net/url_request/url_request_job.cc
@@ -306,7 +306,7 @@ void URLRequestJob::NotifyReadComplete(int bytes_read) {
     return;
 
   // When notifying the delegate, the delegate can release the request
-  // (and thus release 'this').  After calling to the delgate, we must
+  // (and thus release 'this').  After calling to the delegate, we must
   // check the request pointer to see if it still exists, and return
   // immediately if it has been destroyed.  self_preservation ensures our
   // survival until we can get out of this method.
@@ -327,6 +327,10 @@ void URLRequestJob::NotifyReadComplete(int bytes_read) {
     postfilter_bytes_read_ += bytes_read;
     request_->delegate()->OnReadCompleted(request_, bytes_read);
   }
+  VLOG(1) << __FUNCTION__ << "() "
+          << "\"" << (request_ ? request_->url().spec() : "???") << "\""
+          << " bytes read = " << bytes_read
+          << " total = " << postfilter_bytes_read_;

[Randy Smith (Not in Mondays), 2011/05/03 21:30:41]

This seems way too chatty; if someone does --v=1, every time a URLRequest read
completes we'll get a log entry?

[ahendrickson, 2011/05/04 15:08:25]

Oops, that was an oversight.  Changed it to VLOG(20).

 }
 
 void URLRequestJob::NotifyStartError(const URLRequestStatus &status) {