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

Unified Diff: net/quic/chromium/quic_stream_factory.cc

Issue 2699003002: Clear cached QUIC network stats when a QUIC handshake fails. (Closed)
Patch Set: Created 3 years, 10 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
Index: net/quic/chromium/quic_stream_factory.cc
diff --git a/net/quic/chromium/quic_stream_factory.cc b/net/quic/chromium/quic_stream_factory.cc
index 6e63b9be0094e4afb3d28ec36b8d335f6c21bab5..d546a7da03024f096baae084b0b8175032426754 100644
--- a/net/quic/chromium/quic_stream_factory.cc
+++ b/net/quic/chromium/quic_stream_factory.cc
@@ -1947,17 +1947,19 @@ void QuicStreamFactory::ProcessGoingAwaySession(
const QuicConnectionStats& stats = session->connection()->GetStats();
const AlternativeService alternative_service(kProtoQUIC,
server_id.host_port_pair());
+ url::SchemeHostPort server("https", server_id.host_port_pair().host(),
+ server_id.host_port_pair().port());
if (session->IsCryptoHandshakeConfirmed()) {
http_server_properties_->ConfirmAlternativeService(alternative_service);
ServerNetworkStats network_stats;
network_stats.srtt = base::TimeDelta::FromMicroseconds(stats.srtt_us);
network_stats.bandwidth_estimate = stats.estimated_bandwidth;
- url::SchemeHostPort server("https", server_id.host_port_pair().host(),
- server_id.host_port_pair().port());
http_server_properties_->SetServerNetworkStats(server, network_stats);
return;
}
+ http_server_properties_->ClearServerNetworkStats(server);
+
UMA_HISTOGRAM_COUNTS("Net.QuicHandshakeNotConfirmedNumPacketsReceived",
stats.packets_received);

Powered by Google App Engine
This is Rietveld 408576698