Check that ExecutionContext and LocalFrame pointers are not null before getting shouldBypassMainWor…

Check that ExecutionContext and LocalFrame pointers are not null before getting shouldBypassMainWorldCSP via those pointers

This is fix for the case of WebSocket, but EventSource is also fixed as a
bonus.

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

[tyoshino (SeeGerritForStatus), 2014-12-09 05:46:38]

tyoshino@chromium.org changed reviewers:
+ mkwst@chromium.org

[Mike West, 2014-12-09 08:34:02]

The CQ bit was checked by mkwst@chromium.org

[Mike West, 2014-12-09 08:34:03]

LGTM!

[Mike West, 2014-12-09 08:34:16]

The CQ bit was unchecked by mkwst@chromium.org

[Mike West, 2014-12-09 08:34:38]

Actually, before I throw this to the CQ for you, is it possible to add a test
which verifies that we no longer crash here?

[tyoshino (SeeGerritForStatus), 2014-12-17 09:23:31]

On 2014/12/09 08:34:38, Mike West wrote:
> Actually, before I throw this to the CQ for you, is it possible to add a test
> which verifies that we no longer crash here?

Added layout tests.

There I found some other issues with detached frame in
DocumentWebSocketChannel. This CL is blocked by
https://codereview.chromium.org/806423002/

[Mike West, 2014-12-17 09:37:03]

On 2014/12/17 09:23:31, tyoshino wrote:
> On 2014/12/09 08:34:38, Mike West wrote:
> > Actually, before I throw this to the CQ for you, is it possible to add a
test
> > which verifies that we no longer crash here?
> 
> Added layout tests.
> 
> There I found some other issues with detached frame in
> DocumentWebSocketChannel. This CL is blocked by
> https://codereview.chromium.org/806423002/

Thanks! I've taken a look at that patch, looks good to me!

[tyoshino (SeeGerritForStatus), 2014-12-17 13:31:28]

The CQ bit was checked by tyoshino@chromium.org

[commit-bot: I haz the power, 2014-12-17 13:32:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/785933005/40001

[commit-bot: I haz the power, 2014-12-17 15:48:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-17 15:48:50]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/35922)

[tyoshino (SeeGerritForStatus), 2014-12-18 06:17:53]

The CQ bit was checked by tyoshino@chromium.org

[commit-bot: I haz the power, 2014-12-18 06:18:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/785933005/40001

[commit-bot: I haz the power, 2014-12-18 07:55:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-18 07:55:39]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/42498)

[tyoshino (SeeGerritForStatus), 2014-12-18 08:02:20]

The CQ bit was checked by tyoshino@chromium.org

[commit-bot: I haz the power, 2014-12-18 08:03:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/785933005/60001

[commit-bot: I haz the power, 2014-12-18 09:12:01]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=187441