Clear cached QUIC network stats when a QUIC handshake fails.

net/http/http_server_properties_impl_unittest.cc

Index: net/http/http_server_properties_impl_unittest.cc
diff --git a/net/http/http_server_properties_impl_unittest.cc b/net/http/http_server_properties_impl_unittest.cc
index ec278578eb2015d6be9a936267346d23b7e97ca0..be611bfb76abda9426278faa3b942b181ffff941 100644
--- a/net/http/http_server_properties_impl_unittest.cc
+++ b/net/http/http_server_properties_impl_unittest.cc
@@ -1110,6 +1110,17 @@ TEST_F(ServerNetworkStatsServerPropertiesTest, SetServerNetworkStats) {
   EXPECT_EQ(NULL, impl_.GetServerNetworkStats(foo_https_server));
 }
 
+TEST_F(ServerNetworkStatsServerPropertiesTest, ClearServerNetworkStats) {
+  ServerNetworkStats stats;
+  stats.srtt = base::TimeDelta::FromMicroseconds(10);
+  stats.bandwidth_estimate = QuicBandwidth::FromBitsPerSecond(100);
+  url::SchemeHostPort foo_https_server("https", "foo", 443);
+  impl_.SetServerNetworkStats(foo_https_server, stats);
+
+  impl_.ClearServerNetworkStats(foo_https_server);

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

Nit: can you do a GetServerNetworkStats() before the clear, and confirm that
it's non-null?

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

I can but that's tested in the previous test, and the TotT guides say not to
have redundant expectations in tests. (In other words, from the previous test we
proved that SetStats() works. We don't need to re-prove it here)

+  EXPECT_EQ(nullptr, impl_.GetServerNetworkStats(foo_https_server));
+}
+
 typedef HttpServerPropertiesImplTest QuicServerInfoServerPropertiesTest;
 
 TEST_F(QuicServerInfoServerPropertiesTest, Set) {