Fix worker WebSocket crash caused by dereferencing weak pointer

Fix worker WebSocket crash caused by dereferencing weak pointer

WorkerWebSocketChannel::Peer::m_bridge is bound to workerGlobalScopeDidError
as |bridge|. Because it is a CrossThreadWeakPersistent, it's possible that
|m_bridge| is not null when bound but |bridge| is null when called. This CL
adds null pointer checks to avoid a crash.

BUG=625649
Committed: https://crrev.com/bb4da95f6202839ccf81445d1d0ff2cb9c07118b
Cr-Commit-Position: refs/heads/master@{#403744}

[yhirano, 2016-07-05 04:59:52]

yhirano@chromium.org changed reviewers:
+ nhiroki@chromium.org

[yhirano, 2016-07-05 04:59:53]

(context: https://codereview.chromium.org/2025783002/)

[yhirano, 2016-07-05 05:00:55]

Description was changed from

==========
Fix worker WebSocket crash caused by dereferencing weak pointer

WorkerWebSocketChannel::Peer::m_bridge is bound to workerGlobalScopeDidError
as |bridge|. Because it is a CrossThreadWeakPersistent, it's possible that
|m_bridge| is not null when bound but |bridge| is null when called. This CL
adds a null pointer check to avoid a crash.

BUG=625649
==========

to

==========
Fix worker WebSocket crash caused by dereferencing weak pointer

WorkerWebSocketChannel::Peer::m_bridge is bound to workerGlobalScopeDidError
as |bridge|. Because it is a CrossThreadWeakPersistent, it's possible that
|m_bridge| is not null when bound but |bridge| is null when called. This CL
adds null pointer checks to avoid a crash.

BUG=625649
==========

[nhiroki, 2016-07-05 05:11:19]

Thank you for fixing this! LGTM

[yhirano, 2016-07-05 05:14:06]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2016-07-05 05:14:18]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-05 06:09:10]

Description was changed from

==========
Fix worker WebSocket crash caused by dereferencing weak pointer

WorkerWebSocketChannel::Peer::m_bridge is bound to workerGlobalScopeDidError
as |bridge|. Because it is a CrossThreadWeakPersistent, it's possible that
|m_bridge| is not null when bound but |bridge| is null when called. This CL
adds null pointer checks to avoid a crash.

BUG=625649
==========

to

==========
Fix worker WebSocket crash caused by dereferencing weak pointer

WorkerWebSocketChannel::Peer::m_bridge is bound to workerGlobalScopeDidError
as |bridge|. Because it is a CrossThreadWeakPersistent, it's possible that
|m_bridge| is not null when bound but |bridge| is null when called. This CL
adds null pointer checks to avoid a crash.

BUG=625649
==========

[commit-bot: I haz the power, 2016-07-05 06:09:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-05 06:11:06]

Description was changed from

==========
Fix worker WebSocket crash caused by dereferencing weak pointer

WorkerWebSocketChannel::Peer::m_bridge is bound to workerGlobalScopeDidError
as |bridge|. Because it is a CrossThreadWeakPersistent, it's possible that
|m_bridge| is not null when bound but |bridge| is null when called. This CL
adds null pointer checks to avoid a crash.

BUG=625649
==========

to

==========
Fix worker WebSocket crash caused by dereferencing weak pointer

WorkerWebSocketChannel::Peer::m_bridge is bound to workerGlobalScopeDidError
as |bridge|. Because it is a CrossThreadWeakPersistent, it's possible that
|m_bridge| is not null when bound but |bridge| is null when called. This CL
adds null pointer checks to avoid a crash.

BUG=625649

Committed: https://crrev.com/bb4da95f6202839ccf81445d1d0ff2cb9c07118b
Cr-Commit-Position: refs/heads/master@{#403744}
==========

[commit-bot: I haz the power, 2016-07-05 06:11:08]

Patchset 2 (id:??) landed as
https://crrev.com/bb4da95f6202839ccf81445d1d0ff2cb9c07118b
Cr-Commit-Position: refs/heads/master@{#403744}