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..b8f26910f505dfce54f2fe689b7422cf20a3f928 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(20) << __FUNCTION__ << "() "

[darin (slow to review), 2011/05/16 17:47:16]

would it be more helpful if this were output to the net log instead?  same goes
for other VLOGs.

[ahendrickson, 2011/05/16 19:37:40]

I find it useful to use VLOG, as I have a lot of other debugging information
that uses it (which is not appropriate for net-internals).  This way I don't
have to search in two different locations for the information I want, and it's
serial in time).

I can add a net-internals log statement if you think it would be useful.

+           << "\"" << (request_ ? request_->url().spec() : "???") << "\""
+           << " bytes read = " << bytes_read
+           << " total = " << postfilter_bytes_read_;
 }
 
 void URLRequestJob::NotifyStartError(const URLRequestStatus &status) {