Add --enable-websocket-over-spdy command-line flag

net/websockets/websocket_job.cc

Index: net/websockets/websocket_job.cc
diff --git a/net/websockets/websocket_job.cc b/net/websockets/websocket_job.cc
index 95060c693248c06748472dac80a14d0d443ecfa4..e6d1ab8ce792d82ef57fee5423b60586e74352b2 100644
--- a/net/websockets/websocket_job.cc
+++ b/net/websockets/websocket_job.cc
@@ -53,11 +53,19 @@ static base::LazyInstance<WebSocketJobInitSingleton> g_websocket_job_init(
 
 namespace net {
 
+bool WebSocketJob::websocket_over_spdy_ = false;
+
 // static
 void WebSocketJob::EnsureInit() {
   g_websocket_job_init.Get();
 }
 
+// static
+void WebSocketJob::EnableWebSocketOverSpdy(bool enable)

[Yuta Kitamura, 2011/05/20 02:59:48]

I'm a bit worried about inter-thread race condition; this code is probably
harmless but we need to make sure it's really okay.

You can probably check out how --use-spdy is handled.

[Takashi Toyoshima, 2011/05/20 05:25:40]

Good point.
Actually, current caller point is after I/O thread start.
It could cause race.

I change the point. I believe the best position is in
InitializeNetworkOptions(), but there we could not handle about:flags. So I move
it just after about:flags handling. I can set this before any other threads run.

+{

[Yuta Kitamura, 2011/05/20 02:59:48]

Opening brace should be put in the same line as the last function arguments.

[Takashi Toyoshima, 2011/05/20 05:25:40]

Done.
Sorry for poor mistake.

+  websocket_over_spdy_ = enable;
+}
+
 WebSocketJob::WebSocketJob(SocketStream::Delegate* delegate)
     : delegate_(delegate),
       state_(INITIALIZED),