Clear cached QUIC network stats when a QUIC handshake fails.

net/http/http_server_properties_manager.cc

Index: net/http/http_server_properties_manager.cc
diff --git a/net/http/http_server_properties_manager.cc b/net/http/http_server_properties_manager.cc
index 4130a57a4872d81799c523ba4d5507efce24e102..520b0b311398ac403f14968e657e71d467a2c695 100644
--- a/net/http/http_server_properties_manager.cc
+++ b/net/http/http_server_properties_manager.cc
@@ -311,6 +311,16 @@ void HttpServerPropertiesManager::SetServerNetworkStats(
     ScheduleUpdatePrefsOnNetworkThread(SET_SERVER_NETWORK_STATS);
 }
 
+void HttpServerPropertiesManager::ClearServerNetworkStats(
+    const url::SchemeHostPort& server) {
+  DCHECK(network_task_runner_->RunsTasksOnCurrentThread());
+  bool need_update =
+      http_server_properties_impl_->GetServerNetworkStats(server) != nullptr;
+  http_server_properties_impl_->ClearServerNetworkStats(server);
+  if (need_update)
+    ScheduleUpdatePrefsOnNetworkThread(CLEAR_SERVER_NETWORK_STATS);

[Jana, 2017/02/17 02:52:12]

What's this for?

[Ryan Hamilton, 2017/02/17 03:41:53]

The server properties manager persists the properties on the prefs thread, but
it only does this if the state actually changes. This method schedules the
persisting and the if() makes sure we only do that if state changed. Does that
help?

[Jana, 2017/02/17 05:42:33]

Gotcha. Ok, makes sense, thanks.

+}
+
 const ServerNetworkStats* HttpServerPropertiesManager::GetServerNetworkStats(
     const url::SchemeHostPort& server) {
   DCHECK(network_task_runner_->RunsTasksOnCurrentThread());