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

chrome/browser/download/download_manager.cc

 // Copyright (c) 2011 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 "chrome/browser/download/download_manager.h"
 
 #include "base/callback.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/path_service.h"
@@ skipping 521 matching lines @@
       download_history_->UpdateEntry(download);
     }
   }
 }
 
 void DownloadManager::OnResponseCompleted(int32 download_id,
                                           int64 size,
                                           int os_error,
                                           const std::string& hash) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  if (os_error == 0) {
+  // Some web sites lie about the size of the data (for example, using gzip
+  // encoding and specifying the size of the expanded data) and disconnect
+  // when the data is done.  Unfortunately, this means that if we get a
+  // disconnect and the data hasn't been completely downloaded we can't tell.
+  // See http://code.google.com/p/chromium/issues/detail?id=78769.
+  if ((os_error == 0) || (os_error == net::ERR_CONNECTION_CLOSED)) {

[cbentzel, 2011/04/21 17:16:41]

We discussed doing this at a lower-layer [in HttpStreamRequest] only in the case
when the response has Connection: close, and not returning ERR_CONNECTION_CLOSED
in that case even if bytes read in body did not equal Content-Length. There's
likely other cases which will result in ERR_CONNECTION_CLOSED here [i.e. it's a
Keep-Alive connection but there's a TCP RST].

You should take a look at the Firefox code to see how they handle that case as
well.

     OnAllDataSaved(download_id, size, hash);
   } else {
     OnDownloadError(download_id, size, os_error);
   }
 }
 
 void DownloadManager::OnAllDataSaved(int32 download_id,
                                      int64 size,
                                      const std::string& hash) {
   VLOG(20) << __FUNCTION__ << "()" << " download_id = " << download_id
@@ skipping 621 matching lines @@
     observed_download_manager_->RemoveObserver(this);
 }
 
 void DownloadManager::OtherDownloadManagerObserver::ModelChanged() {
   observing_download_manager_->NotifyModelChanged();
 }
 
 void DownloadManager::OtherDownloadManagerObserver::ManagerGoingDown() {
   observed_download_manager_ = NULL;
 }