Change QuicStreamRequest::Request() to take a preferred QuicVersion so that

net/http/http_server_properties.h

Index: net/http/http_server_properties.h
diff --git a/net/http/http_server_properties.h b/net/http/http_server_properties.h
index a11d80222f239b32dcb26f36bfdfb6472df487f3..961544fea01ca83319feb47042cf1873e77c682a 100644
--- a/net/http/http_server_properties.h
+++ b/net/http/http_server_properties.h
@@ -175,6 +175,12 @@ class NET_EXPORT_PRIVATE AlternativeServiceInfo {
     return alternative_service_;
   }
 
+  const NextProto& protocol() const { return alternative_service_.protocol; }
+
+  HostPortPair host_port_pair() const {

[Ryan Hamilton, 2017/06/28 18:53:14]

Should this be a const& to avoid the (possible) string copy?

[Zhongyi Shi, 2017/07/05 23:08:59]

alternative_service_.host_port_pair() is a local temporary copy. There is no
HostPort field in AlternativeService.

+    return alternative_service_.host_port_pair();
+  }
+
   base::Time expiration() const { return expiration_; }
 
   const QuicVersionVector& advertised_versions() const {