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 754372bb88aec3549c08266e07992ed455b8bce3..c394033f9d43edb9ce061e69cf9f0a45c4c89cef 100644
--- a/net/http/http_stream_factory.cc
+++ b/net/http/http_stream_factory.cc
@@ -86,7 +86,7 @@ void HttpStreamFactory::ProcessAlternateProtocol(
   if (mapping_rules)
     mapping_rules->RewriteHost(&host_port);
 
-  http_server_properties->SetAlternateProtocol(
+  http_server_properties->AddAlternateProtocol(
       host_port, static_cast<uint16>(port), protocol, probability);

[Ryan Hamilton, 2015/02/27 18:20:48]

I don't think this is quite right. Let's say that initially we have:
  Alternate-Protocol: QUIC:443
But then we see:
  Alternate-Protocol: npn-spdy:80
(This example is lame, but bear with me :>). I think that as written, this code
will end up with *both* mappings active, but I think we only want one. What do
you think?

[Bence, 2015/02/27 19:31:41]

You are right.  So we will have another method for parsing alternative services,
right?  And we want both ProcessAlternateProtocols and ProcessAlternateServices
to overwrite existing settings, right?

Done.

 }