Add an option to disable net::BidirectionalStreamQuicImpl

net/http/http_network_session.h

Index: net/http/http_network_session.h
diff --git a/net/http/http_network_session.h b/net/http/http_network_session.h
index 30cec77f04d48d0ec4aad019a6abd4dece195f9b..dce15ec59a67c91649fc0c1693a9b894e9ca5c2c 100644
--- a/net/http/http_network_session.h
+++ b/net/http/http_network_session.h
@@ -185,6 +185,10 @@ class NET_EXPORT HttpNetworkSession
     // If true, active QUIC sessions experiencing poor connectivity may be
     // migrated onto a new network.
     bool quic_migrate_sessions_early;
+    // If false, net::BidirectionalStreamQuicImpl will not be used when

[mef, 2016/03/15 19:51:10]

I think it should say 'If true'...

[xunjieli, 2016/03/15 20:15:20]

Done.

[Ryan Hamilton, 2016/03/15 22:41:44]

I think a better comment might be:

// If true, then bidirectional stream over QUIC will be disabled. 

[xunjieli, 2016/03/16 15:47:04]

Done.

+    // requesting a net::BidirectionalStream.
+    bool disable_bidirectional_stream_quic_impl;

[mef, 2016/03/15 19:51:10]

should we call it 'quic_disable_bidirectional_stream'?

[xunjieli, 2016/03/15 20:15:20]

I am not sure. It isn't a feature of QUIC (but a flag on the bidi wrapper on top
of QUIC). I am not sure if it should be lumped together with other quic feature
flags above.

[Ryan Hamilton, 2016/03/15 22:41:44]

I like mef's name.

[xunjieli, 2016/03/16 15:47:04]

Done. Great!

+
     ProxyDelegate* proxy_delegate;
     // Enable support for Token Binding.
     bool enable_token_binding;