Index: net/quic/chromium/quic_chromium_client_session.cc |
diff --git a/net/quic/chromium/quic_chromium_client_session.cc b/net/quic/chromium/quic_chromium_client_session.cc |
index 27818a5c50cfbdd00cc840b36a94c1fc18b1690e..6913eb51a58ff40bc801666e544a66c03604248e 100644 |
--- a/net/quic/chromium/quic_chromium_client_session.cc |
+++ b/net/quic/chromium/quic_chromium_client_session.cc |
@@ -375,24 +375,8 @@ QuicChromiumClientSession::~QuicChromiumClientSession() { |
round_trip_handshakes, 1, 3, 4); |
} |
} |
+ |
const QuicConnectionStats stats = connection()->GetStats(); |
- if (server_info_ && stats.min_rtt_us > 0) { |
- base::TimeTicks wait_for_data_start_time = |
- server_info_->wait_for_data_start_time(); |
- base::TimeTicks wait_for_data_end_time = |
- server_info_->wait_for_data_end_time(); |
- if (!wait_for_data_start_time.is_null() && |
- !wait_for_data_end_time.is_null()) { |
- base::TimeDelta wait_time = |
- wait_for_data_end_time - wait_for_data_start_time; |
- const base::HistogramBase::Sample kMaxWaitToRtt = 1000; |
- base::HistogramBase::Sample wait_to_rtt = |
- static_cast<base::HistogramBase::Sample>( |
- 100 * wait_time.InMicroseconds() / stats.min_rtt_us); |
- UMA_HISTOGRAM_CUSTOM_COUNTS("Net.QuicServerInfo.WaitForDataReadyToRtt", |
- wait_to_rtt, 1, kMaxWaitToRtt, 50); |
- } |
- } |
// The MTU used by QUIC is limited to a fairly small set of predefined values |
// (initial values and MTU discovery values), but does not fare well when |
@@ -862,21 +846,6 @@ void QuicChromiumClientSession::OnCryptoHandshakeEvent( |
UMA_HISTOGRAM_TIMES( |
"Net.QuicSession.HandshakeConfirmedTime", |
connect_timing_.connect_end - connect_timing_.connect_start); |
- |
- if (server_info_) { |
- // TODO(rtenneti): Should we delete this histogram? |
- // Track how long it has taken to finish handshake once we start waiting |
- // for reading of QUIC server information from disk cache. We could use |
- // this data to compare total time taken if we were to cancel the disk |
- // cache read vs waiting for the read to complete. |
- base::TimeTicks wait_for_data_start_time = |
- server_info_->wait_for_data_start_time(); |
- if (!wait_for_data_start_time.is_null()) { |
- UMA_HISTOGRAM_TIMES( |
- "Net.QuicServerInfo.WaitForDataReady.HandshakeConfirmedTime", |
- base::TimeTicks::Now() - wait_for_data_start_time); |
- } |
- } |
// Track how long it has taken to finish handshake after we have finished |
// DNS host resolution. |
if (!connect_timing_.dns_end.is_null()) { |
@@ -891,8 +860,6 @@ void QuicChromiumClientSession::OnCryptoHandshakeEvent( |
++it; |
observer->OnCryptoHandshakeConfirmed(); |
} |
- if (server_info_) |
- server_info_->OnExternalCacheHit(); |
} |
QuicSpdySession::OnCryptoHandshakeEvent(event); |
} |