Enable SPDY 3.1 and 3.0 in UrlRequestContextBuilder by default

net/url_request/url_request_context_builder.cc

Index: net/url_request/url_request_context_builder.cc
diff --git a/net/url_request/url_request_context_builder.cc b/net/url_request/url_request_context_builder.cc
index 7b2c9534bb2e1cb6377bc791f80f338b62dfbb74..5fb588038b361d444fdf0d54b9b203efab00d566 100644
--- a/net/url_request/url_request_context_builder.cc
+++ b/net/url_request/url_request_context_builder.cc
@@ -176,7 +176,13 @@ URLRequestContextBuilder::HttpNetworkSessionParams::HttpNetworkSessionParams()
       host_mapping_rules(NULL),
       testing_fixed_http_port(0),
       testing_fixed_https_port(0),
-      trusted_spdy_proxy() {}
+      use_alternate_protocols(true) {
+  // By default, support SPDY and enable NPN.  These are different defaults from
+  // HttpNetworkSession.
+  next_protos.push_back(kProtoHTTP11);
+  next_protos.push_back(kProtoSPDY3);
+  next_protos.push_back(kProtoSPDY31);

[Ryan Hamilton, 2014/05/29 19:54:38]

So at some point, we're going to turn QUIC on by default. I'm sure I won't
remember to do it here :> In the least, I'd be inclined to use one of the
existing NextProtos methods from next_protos.h. But I think we really should
work towards having the right defaults. So I think your suggestion of doing this
in HttpNetworkSession::Params is the right way forward. I you want to land this
while you work on that, I'm fine with that approach, but I wouldn't want to see
this code exist as anything other than a bandaid. What do you think?

+}
 
 URLRequestContextBuilder::HttpNetworkSessionParams::~HttpNetworkSessionParams()
 {}
@@ -277,6 +283,7 @@ URLRequestContext* URLRequestContextBuilder::Build() {
   network_session_params.http_server_properties =
       context->http_server_properties();
   network_session_params.net_log = context->net_log();
+
   network_session_params.ignore_certificate_errors =
       http_network_session_params_.ignore_certificate_errors;
   network_session_params.host_mapping_rules =
@@ -285,8 +292,11 @@ URLRequestContext* URLRequestContextBuilder::Build() {
       http_network_session_params_.testing_fixed_http_port;
   network_session_params.testing_fixed_https_port =
       http_network_session_params_.testing_fixed_https_port;
+  network_session_params.use_alternate_protocols =
+    http_network_session_params_.use_alternate_protocols;
   network_session_params.trusted_spdy_proxy =
       http_network_session_params_.trusted_spdy_proxy;
+  network_session_params.next_protos = http_network_session_params_.next_protos;
 
   HttpTransactionFactory* http_transaction_factory = NULL;
   if (http_cache_enabled_) {