Clear cached QUIC network stats when a QUIC handshake fails.

net/http/http_server_properties_manager.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/http/http_server_properties_manager.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/memory/ptr_util.h"
@@ skipping 293 matching lines @@
       http_server_properties_impl_->GetServerNetworkStats(server);
   if (http_server_properties_impl_->GetServerNetworkStats(server))
     old_stats = *old_stats_ptr;
   http_server_properties_impl_->SetServerNetworkStats(server, stats);
   ServerNetworkStats new_stats =
       *(http_server_properties_impl_->GetServerNetworkStats(server));
   if (old_stats != new_stats)
     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());
   return http_server_properties_impl_->GetServerNetworkStats(server);
 }
 
 const ServerNetworkStatsMap&
 HttpServerPropertiesManager::server_network_stats_map() const {
   DCHECK(network_task_runner_->RunsTasksOnCurrentThread());
   return http_server_properties_impl_->server_network_stats_map();
@@ skipping 798 matching lines @@
   if (!setting_prefs_)
     ScheduleUpdateCacheOnPrefThread();
 }
 
 void HttpServerPropertiesManager::SetInitialized() {
   DCHECK(network_task_runner_->RunsTasksOnCurrentThread());
   is_initialized_ = true;
 }
 
 }  // namespace net