Set alternative service even if it is broken.

net/http/http_server_properties_impl_unittest.cc

 // Copyright (c) 2012 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_impl.h"
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 347 matching lines @@
   ASSERT_TRUE(HasAlternativeService(test_host_port_pair));
   EXPECT_TRUE(impl_.IsAlternativeServiceBroken(alternative_service1));
 
   const AlternativeService alternative_service2(NPN_SPDY_4, "foo", 1234);
   impl_.SetAlternativeService(test_host_port_pair, alternative_service2, 1.0);
   EXPECT_TRUE(impl_.IsAlternativeServiceBroken(alternative_service1));
   EXPECT_FALSE(impl_.IsAlternativeServiceBroken(alternative_service2));
   EXPECT_EQ(1234, impl_.GetAlternativeService(test_host_port_pair).port);
 }
 
+// A broken alternative service in the mapping carries meaningful information,
+// therefore it should not be ignored by SetAlternativeService().  In
+// particular, an alternative service mapped to an origin shadows alternative
+// services of canonical hosts.
+TEST_F(AlternateProtocolServerPropertiesTest, BrokenShadowsCanonical) {
+  HostPortPair test_host_port_pair("foo.c.youtube.com", 80);
+  HostPortPair canonical_port_pair("bar.c.youtube.com", 80);
+  AlternativeService canonical_altsvc(QUIC, "bar.c.youtube.com", 1234);
+  impl_.SetAlternativeService(canonical_port_pair, canonical_altsvc, 1.0);
+  EXPECT_TRUE(impl_.GetAlternativeService(test_host_port_pair) ==
+              canonical_altsvc);
+
+  const AlternativeService broken_alternative_service(NPN_SPDY_4, "foo", 443);
+  impl_.MarkAlternativeServiceBroken(broken_alternative_service);
+  EXPECT_TRUE(impl_.IsAlternativeServiceBroken(broken_alternative_service));
+
+  impl_.SetAlternativeService(test_host_port_pair, broken_alternative_service,
+                              1.0);
+  ASSERT_TRUE(impl_.GetAlternativeService(test_host_port_pair) ==

[Ryan Hamilton, 2015/06/24 21:08:32]

nit: Does ASSERT_EQ() work?

[Bence, 2015/06/25 11:24:34]

Oops, it does work.  Sorry, I thought I need to introduce operator<< for logging
for that, but it turns out it just prints out binary blobs if the test fails. 
That being said, defining operator<< would be useful in a later CL.

Done.

+              broken_alternative_service);
+}

[Ryan Hamilton, 2015/06/24 21:08:32]

Can you also verify that:

EXPECT_TRUE(impl_.IsAlternativeServiceBroken(broken_alternative_service));

[Bence, 2015/06/25 11:24:34]

Done.

+
 TEST_F(AlternateProtocolServerPropertiesTest, ClearBroken) {
   HostPortPair test_host_port_pair("foo", 80);
   const AlternativeService alternative_service(NPN_SPDY_4, "foo", 443);
   impl_.SetAlternativeService(test_host_port_pair, alternative_service, 1.0);
   impl_.MarkAlternativeServiceBroken(alternative_service);
   ASSERT_TRUE(HasAlternativeService(test_host_port_pair));
   EXPECT_TRUE(impl_.IsAlternativeServiceBroken(alternative_service));
   impl_.ClearAlternativeService(test_host_port_pair);
   EXPECT_TRUE(impl_.IsAlternativeServiceBroken(alternative_service));
 }
@@ skipping 411 matching lines @@
   EXPECT_EQ(100, stats2->bandwidth_estimate.ToBitsPerSecond());
 
   impl_.Clear();
   const ServerNetworkStats* stats3 = impl_.GetServerNetworkStats(foo_server);
   EXPECT_EQ(NULL, stats3);
 }
 
 }  // namespace
 
 }  // namespace net