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

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

m_syncHelper will be 0 if initialize() fails. connect() must just return
with false in such case.

BUG=368329
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=172973

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

https://codereview.chromium.org/266483003/diff/1/Source/modules/websockets/Wo...
File Source/modules/websockets/WorkerThreadableWebSocketChannel.cpp (right):

https://codereview.chromium.org/266483003/diff/1/Source/modules/websockets/Wo...
Source/modules/websockets/WorkerThreadableWebSocketChannel.cpp:493: if
(!m_workerGlobalScope)
m_syncHelper?

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

https://codereview.chromium.org/266483003/diff/1/Source/modules/websockets/Wo...
File Source/modules/websockets/WorkerThreadableWebSocketChannel.cpp (right):

https://codereview.chromium.org/266483003/diff/1/Source/modules/websockets/Wo...
Source/modules/websockets/WorkerThreadableWebSocketChannel.cpp:493: if
(!m_workerGlobalScope)
On 2014/04/30 07:17:08, yhirano wrote:
> m_syncHelper?

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.

[yhirano, 2014-04-30 08:14:24]

lgtm

[tyoshino (SeeGerritForStatus), 2014-04-30 10:45:20]

The CQ bit was checked by tyoshino@chromium.org

[commit-bot: I haz the power, 2014-04-30 10:45:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tyoshino@chromium.org/266483003/1

[commit-bot: I haz the power, 2014-04-30 11:46:41]

Change committed as 172973