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

content/browser/renderer_host/socket_stream_dispatcher_host.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 "content/browser/renderer_host/socket_stream_dispatcher_host.h"
 
 #include "base/logging.h"
 #include "chrome/browser/profiles/profile.h"
 #include "content/browser/renderer_host/socket_stream_host.h"
 #include "content/common/socket_stream.h"
@@ skipping 50 matching lines @@
     DeleteSocketStreamHost(socket_id);
   }
 }
 
 void SocketStreamDispatcherHost::OnSentData(net::SocketStream* socket,
                                             int amount_sent) {
   int socket_id = SocketStreamHost::SocketIdFromSocketStream(socket);
   DVLOG(1) << "SocketStreamDispatcherHost::OnSentData socket_id=" << socket_id
            << " amount_sent=" << amount_sent;
   if (socket_id == content_common::kNoSocketId) {
-    LOG(ERROR) << "NoSocketId in OnReceivedData";
+    LOG(ERROR) << "NoSocketId in OnSentData";

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

Ditto.  This is a fine fix, but I don't think it belongs as part of this CL.

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

If I don't do it in this CL, I'll forget about it.  If someone else wants to do
it, I'm fine with that.

     return;
   }
   if (!Send(new SocketStreamMsg_SentData(socket_id, amount_sent))) {
     LOG(ERROR) << "SocketStreamMsg_SentData failed.";
     DeleteSocketStreamHost(socket_id);
   }
 }
 
 void SocketStreamDispatcherHost::OnReceivedData(
     net::SocketStream* socket, const char* data, int len) {
@@ skipping 65 matching lines @@
   hosts_.Remove(socket_id);
   if (!Send(new SocketStreamMsg_Closed(socket_id))) {
     LOG(ERROR) << "SocketStreamMsg_Closed failed.";
   }
 }
 
 net::URLRequestContext* SocketStreamDispatcherHost::GetURLRequestContext() {
   return url_request_context_selector_->GetRequestContext(
       ResourceType::SUB_RESOURCE);
 }