[WebSocket] Do nothing for connect() if initialization has failed.

Source/modules/websockets/WorkerThreadableWebSocketChannel.cpp

Index: Source/modules/websockets/WorkerThreadableWebSocketChannel.cpp
diff --git a/Source/modules/websockets/WorkerThreadableWebSocketChannel.cpp b/Source/modules/websockets/WorkerThreadableWebSocketChannel.cpp
index 705b99d0087a20c063f983878c68d6e16aa7233e..3fb04f773687d1114e736fddbb2420ac7e70cde2 100644
--- a/Source/modules/websockets/WorkerThreadableWebSocketChannel.cpp
+++ b/Source/modules/websockets/WorkerThreadableWebSocketChannel.cpp
@@ -490,6 +490,9 @@ void WorkerThreadableWebSocketChannel::Bridge::initialize(const String& sourceUR
 bool WorkerThreadableWebSocketChannel::Bridge::connect(const KURL& url, const String& protocol)
 {
     ASSERT(m_workerClientWrapper);
+    if (!m_workerGlobalScope)

[yhirano, 2014/04/30 07:17:08]

m_syncHelper?

[tyoshino (SeeGerritForStatus), 2014/04/30 07:36:23]

m_workerGlobalScope can also tell whether initialize() was successful or not.
I'd like to keep this consistent with the rest of methods below and refactor
later. Re: m_workerClientWrapper, I already have CL for removing it so let me
omit only m_workerClientWrapper.

+        return false;
+    ASSERT(m_syncHelper);
     m_loaderProxy.postTaskToLoader(CallClosureTask::create(bind(&Peer::connect, m_peer, url.copy(), protocol.isolatedCopy())));
     RefPtr<Bridge> protect(this);
     waitForMethodCompletion();