ABANDONED Handle multiple AlternateProtocols for each HostPortPair.

net/http/http_stream_factory.cc

Index: net/http/http_stream_factory.cc
diff --git a/net/http/http_stream_factory.cc b/net/http/http_stream_factory.cc
index e86d8f89b071a9216854512694d174aaa85475a2..cdaf94f7d13034ec94fc4b3841eece7956fe552e 100644
--- a/net/http/http_stream_factory.cc
+++ b/net/http/http_stream_factory.cc
@@ -90,16 +90,8 @@ void HttpStreamFactory::ProcessAlternateProtocol(
   if (mapping_rules)
     mapping_rules->RewriteHost(&host_port);
 
-  if (http_server_properties->HasAlternateProtocol(host_port)) {
-    const AlternateProtocolInfo existing_alternate =
-        http_server_properties->GetAlternateProtocol(host_port);
-    // If we think the alternate protocol is broken, don't change it.
-    if (existing_alternate.protocol == ALTERNATE_PROTOCOL_BROKEN)
-      return;
-  }
-
-  http_server_properties->SetAlternateProtocol(host_port, port, protocol,
-                                               probability);
+  http_server_properties->AddAlternateProtocol(
+      host_port, port, protocol, probability);

[Ryan Hamilton, 2014/11/03 19:00:30]

Does this need to Clear() first? Otherwise it seems like it will simply append?

[Bence, 2014/11/03 22:32:26]

Previous behavior is that if an alternate protocol was known to be broken, we
don't overwrite it.  This can be conveniently mimiced this way.

BTW Alternate protocol and alternate service headers are going to be cumulative,
are they not?

[Ryan Hamilton, 2014/11/04 18:56:22]

In the case of non-broken, the old code overwrite the existing entry with the
new entry. With this new code, it seems like we will end up appending, right? I
don't think that's desirable.
In the sense that we should union them together? Possibly. Although I was
talking with alyssa and perhaps it makes more sense to use Alt-Svc exclusively
if it exists. And only use Alternate-Protocol if Alt-Svc does not exist.

[Bence, 2014/11/04 21:55:50]

Done.  FYI I had to create a new method in HttpServerProperties*, because
GetAlternateProtocol fails if there is none, and HasAlternateProtocol returns
false if they are all below the probability threshold.

 }
 
 GURL HttpStreamFactory::ApplyHostMappingRules(const GURL& url,