JobController: do not create alternative job for those AlternativeServiceInfo

net/http/http_stream_factory_impl_job_controller.h

Index: net/http/http_stream_factory_impl_job_controller.h
diff --git a/net/http/http_stream_factory_impl_job_controller.h b/net/http/http_stream_factory_impl_job_controller.h
index 76a70e1aa7333e516c6b55f161925f6d3374f44d..e7b41816944ce9b8687d8ffbbcbc6c7691d280ea 100644
--- a/net/http/http_stream_factory_impl_job_controller.h
+++ b/net/http/http_stream_factory_impl_job_controller.h
@@ -265,6 +265,13 @@ class HttpStreamFactoryImpl::JobController
       HttpStreamRequest::Delegate* delegate,
       HttpStreamRequest::StreamType stream_type);
 
+  // Returns a QuicVersion that has been advertised in |advertised_versions|
+  // and supported by the net stack.  If more than one QuicVersions are

[Ryan Hamilton, 2017/06/30 02:45:11]

nit: can you remove "by the net stack" from this comment (and elsewhere).

... " and is supported."

[Zhongyi Shi, 2017/06/30 21:08:43]

Done.

+  // supported, the first matched in the supported versions will be returned.
+  // If no mutually supported version is found,
+  // QUIC_VERSION_UNSUPPORTED_VERSION will be returned.
+  QuicVersion SelectQuicVersion(const QuicVersionVector& advertised_versions);
+
   // Remove session from the SpdySessionRequestMap.
   void RemoveRequestFromSpdySessionRequestMap();