Independently enable SPDY versions from field trial.

net/socket/next_proto.cc

 // Copyright 2014 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/socket/next_proto.h"
 
 namespace net {
 
-NextProtoVector NextProtosHttpOnly() {
+NextProtoVector NextProtos(bool quic_enabled,
+                           bool spdy31_enabled,
+                           bool h2_14_enabled,
+                           bool h2_enabled) {
   NextProtoVector next_protos;
   next_protos.push_back(kProtoHTTP11);
+  if (quic_enabled)
+    next_protos.push_back(kProtoQUIC1SPDY3);
+  if (spdy31_enabled)
+    next_protos.push_back(kProtoSPDY31);
+  if (h2_14_enabled)
+    next_protos.push_back(kProtoSPDY4_14);
+  if (h2_enabled)
+    next_protos.push_back(kProtoSPDY4);
   return next_protos;
 }
 
 NextProtoVector NextProtosDefaults() {
-  NextProtoVector next_protos;
-  next_protos.push_back(kProtoHTTP11);
-  next_protos.push_back(kProtoSPDY31);
-  next_protos.push_back(kProtoSPDY4_14);
-  next_protos.push_back(kProtoSPDY4);
-  return next_protos;
+  return NextProtos(false, true, true, true);
+}
+
+NextProtoVector NextProtosSpdy31() {
+  return NextProtos(true, true, false, false);

[Ryan Hamilton, 2015/03/20 23:38:07]

Lots of booleans is a recipe for typos. How about just taking a NextProtoVector
directly? (Perhaps performing validation on it that it only contains the protos
you would like the caller to be able to use?

[Bence, 2015/03/23 14:19:13]

Okay, you convinced me.  I'm moving the logic to io_thread.cc.

Done.

 }
 
 NextProtoVector NextProtosWithSpdyAndQuic(bool spdy_enabled,
                                           bool quic_enabled) {
-  NextProtoVector next_protos;
-  next_protos.push_back(kProtoHTTP11);
-  if (quic_enabled)
-    next_protos.push_back(kProtoQUIC1SPDY3);
-  if (spdy_enabled) {
-    next_protos.push_back(kProtoSPDY31);
-    next_protos.push_back(kProtoSPDY4_14);
-    next_protos.push_back(kProtoSPDY4);
-  }
-  return next_protos;
-}
-
-NextProtoVector NextProtosSpdy31() {
-  NextProtoVector next_protos;
-  next_protos.push_back(kProtoHTTP11);
-  next_protos.push_back(kProtoQUIC1SPDY3);
-  next_protos.push_back(kProtoSPDY31);
-  return next_protos;
-}
-
-NextProtoVector NextProtosSpdy4Http2() {
-  NextProtoVector next_protos;
-  next_protos.push_back(kProtoHTTP11);
-  next_protos.push_back(kProtoQUIC1SPDY3);
-  next_protos.push_back(kProtoSPDY31);
-  next_protos.push_back(kProtoSPDY4_14);
-  next_protos.push_back(kProtoSPDY4);
-  return next_protos;
+  return NextProtos(spdy_enabled, spdy_enabled, spdy_enabled, quic_enabled);
 }
 
 }  // namespace net