Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1170)

Unified Diff: net/http/http_network_transaction.cc

Issue 174287: Resend on IO errors on late bound sockets that were idle. (Closed)
Patch Set: Update comment. Created 11 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | net/socket/client_socket_handle.h » ('j') | net/socket/client_socket_handle.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_network_transaction.cc
diff --git a/net/http/http_network_transaction.cc b/net/http/http_network_transaction.cc
index 79be223f99dccab0c2891b905933596231ed7781..d37bab482f831e107804e1a904714c17ad3dff3c 100644
--- a/net/http/http_network_transaction.cc
+++ b/net/http/http_network_transaction.cc
@@ -1554,7 +1554,8 @@ bool HttpNetworkTransaction::ShouldResendRequest() const {
// This automatically prevents an infinite resend loop because we'll run
// out of the cached keep-alive connections eventually.
if (establishing_tunnel_ ||
- !reused_socket_ || // We didn't reuse a keep-alive connection.
+ // We used a socket that was never idle.
+ connection_.reuse_type() == ClientSocketHandle::UNUSED ||
header_buf_len_) { // We have received some response headers.
return false;
}
« no previous file with comments | « no previous file | net/socket/client_socket_handle.h » ('j') | net/socket/client_socket_handle.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698