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

Side by Side Diff: net/socket/client_socket_pool_base.cc

Issue 6800009: Attn: Mike Belshe Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: Created 9 years, 8 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/socket/client_socket_pool_base.h" 5 #include "net/socket/client_socket_pool_base.h"
6 6
7 #include "base/compiler_specific.h" 7 #include "base/compiler_specific.h"
8 #include "base/format_macros.h" 8 #include "base/format_macros.h"
9 #include "base/message_loop.h" 9 #include "base/message_loop.h"
10 #include "base/metrics/stats_counters.h" 10 #include "base/metrics/stats_counters.h"
(...skipping 10 matching lines...) Expand all
21 namespace { 21 namespace {
22 22
23 // The timeout value, in seconds, used to clean up idle sockets that can't be 23 // The timeout value, in seconds, used to clean up idle sockets that can't be
24 // reused. 24 // reused.
25 // 25 //
26 // Note: It's important to close idle sockets that have received data as soon 26 // Note: It's important to close idle sockets that have received data as soon
27 // as possible because the received data may cause BSOD on Windows XP under 27 // as possible because the received data may cause BSOD on Windows XP under
28 // some conditions. See http://crbug.com/4606. 28 // some conditions. See http://crbug.com/4606.
29 const int kCleanupInterval = 10; // DO NOT INCREASE THIS TIMEOUT. 29 const int kCleanupInterval = 10; // DO NOT INCREASE THIS TIMEOUT.
30 30
31 // Indicate whether or not we should establish a new TCP connection after a 31 // Indicate whether or not we should establish a new transport layer connection
32 // certain timeout has passed without receiving an ACK. 32 // after a certain timeout has passed without receiving an ACK/SACK.
33 bool g_connect_backup_jobs_enabled = true; 33 bool g_connect_backup_jobs_enabled = true;
Mike Belshe 2011/04/06 18:32:53 this comment change is part of the TCP -> Transpor
34 34
35 } // namespace 35 } // namespace
36 36
37 namespace net { 37 namespace net {
38 38
39 ConnectJob::ConnectJob(const std::string& group_name, 39 ConnectJob::ConnectJob(const std::string& group_name,
40 base::TimeDelta timeout_duration, 40 base::TimeDelta timeout_duration,
41 Delegate* delegate, 41 Delegate* delegate,
42 const BoundNetLog& net_log) 42 const BoundNetLog& net_log)
43 : group_name_(group_name), 43 : group_name_(group_name),
(...skipping 1038 matching lines...) Expand 10 before | Expand all | Expand 10 after
1082 // Delete active jobs. 1082 // Delete active jobs.
1083 STLDeleteElements(&jobs_); 1083 STLDeleteElements(&jobs_);
1084 1084
1085 // Cancel pending backup job. 1085 // Cancel pending backup job.
1086 method_factory_.RevokeAll(); 1086 method_factory_.RevokeAll();
1087 } 1087 }
1088 1088
1089 } // namespace internal 1089 } // namespace internal
1090 1090
1091 } // namespace net 1091 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698