ABANDONED Handle multiple AlternateProtocols for each HostPortPair.

net/http/http_server_properties.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.h"
 
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/stringprintf.h"
 #include "net/socket/ssl_client_socket.h"
@@ skipping 85 matching lines @@
     case kProtoHTTP11:
       break;
   }
 
   NOTREACHED() << "Invalid NextProto: " << next_proto;
   return UNINITIALIZED_ALTERNATE_PROTOCOL;
 }
 
 std::string AlternateProtocolInfo::ToString() const {
   return base::StringPrintf("%d:%s p=%f%s", port,
-                            AlternateProtocolToString(protocol),
-                            probability,
+                            AlternateProtocolToString(protocol), probability,
                             is_broken ? " (broken)" : "");
 }
 
+std::string AlternateProtocolsToString(const AlternateProtocols& protocols) {
+  std::string output{};

[Bence, 2015/01/30 21:24:50]

I don't think this format is currently allowed.

[Ryan Hamilton, 2015/01/30 21:50:19]

the {}s? What's different from simply doing std::string output;?

+  for (AlternateProtocols::const_iterator it = protocols.begin();
+       it != protocols.end(); ++it) {
+    output.append(it->ToString());
+    output.append("; ");
+  }
+  return output;
+}
+
 // static
 void HttpServerProperties::ForceHTTP11(SSLConfig* ssl_config) {
   ssl_config->next_protos.clear();
   ssl_config->next_protos.push_back(kProtoHTTP11);
 }
 
 }  // namespace net