SPDY - disable spdy/2.1 (flow control) by default

net/http/http_stream_factory.h

Index: net/http/http_stream_factory.h
===================================================================
--- net/http/http_stream_factory.h	(revision 124201)
+++ net/http/http_stream_factory.h	(working copy)
@@ -15,6 +15,7 @@
 #include "net/base/completion_callback.h"
 #include "net/base/load_states.h"
 #include "net/base/net_export.h"
+#include "net/http/http_server_properties.h"

[Ryan Hamilton, 2012/03/02 16:46:53]

Is this import needed?

[ramant (doing other things), 2012/03/02 18:40:41]

It is needed because of NUM_ALTERNATE_PROTOCOLS.

 #include "net/socket/ssl_client_socket.h"
 
 class GURL;
@@ -230,11 +231,7 @@
   static bool HasSpdyExclusion(const HostPortPair& endpoint);
 
   // Sets the next protocol negotiation value used during the SSL handshake.

[Ryan Hamilton, 2012/03/02 16:46:53]

Please update comment. I believe this vector is use to negotiate a protocol via
SSL NPN, as well as via HTTP Alternate-Protocol.

[ramant (doing other things), 2012/03/02 18:40:41]

Done.

-  static void set_next_protos(const std::vector<std::string>& value) {
-    if (!next_protos_)
-      next_protos_ = new std::vector<std::string>;
-    *next_protos_ = value;
-  }
+  static void SetNextProtos(const std::vector<std::string>& value);
   static bool has_next_protos() { return next_protos_ != NULL; }
   static const std::vector<std::string>& next_protos() {
     return *next_protos_;
@@ -274,6 +271,7 @@
 
   static const HostMappingRules* host_mapping_rules_;
   static std::vector<std::string>* next_protos_;
+  static bool supported_protocols_[NUM_ALTERNATE_PROTOCOLS];

[Ryan Hamilton, 2012/03/02 16:46:53]

nit: I think I would call this enabled_protocols_, because Chrome "supports" all
of them, but some might be disabled via command line flags.

[ramant (doing other things), 2012/03/02 18:40:41]

Done.

   static bool spdy_enabled_;
   static bool use_alternate_protocols_;
   static bool force_spdy_over_ssl_;