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

Unified Diff: net/socket/transport_client_socket_pool.h

Issue 68793007: Added three histograms to track the following TCP connection latency (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixed jar's comments Created 7 years, 1 month 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/transport_client_socket_pool.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/socket/transport_client_socket_pool.h
diff --git a/net/socket/transport_client_socket_pool.h b/net/socket/transport_client_socket_pool.h
index 95b51c36ce056e9b92f4b40181600d3e9da82263..1c22bf29ec3776886ddb0e3a5d71084819f34477 100644
--- a/net/socket/transport_client_socket_pool.h
+++ b/net/socket/transport_client_socket_pool.h
@@ -93,6 +93,12 @@ class NET_EXPORT_PRIVATE TransportConnectJob : public ConnectJob {
STATE_NONE,
};
+ enum ConnectInterval {
+ CONNECT_INTERVAL_LE_10MS,
+ CONNECT_INTERVAL_LE_20MS,
+ CONNECT_INTERVAL_GT_20MS,
+ };
+
void OnIOComplete(int result);
// Runs the state transition loop.
@@ -125,9 +131,8 @@ class NET_EXPORT_PRIVATE TransportConnectJob : public ConnectJob {
base::TimeTicks fallback_connect_start_time_;
base::OneShotTimer<TransportConnectJob> fallback_timer_;
- // If the interval between this connect and previous connect is less than
- // 20ms, then |less_than_20ms_since_connect_| is set to true.
- bool less_than_20ms_since_connect_;
+ // Track the interval between this connect and previous connect.
+ ConnectInterval interval_between_connects_;
DISALLOW_COPY_AND_ASSIGN(TransportConnectJob);
};
« no previous file with comments | « no previous file | net/socket/transport_client_socket_pool.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698