Permissions: Avoid poking at the heap after Blink shutdown

Permissions: Avoid poking at the heap after Blink shutdown

Changes serviceConnectionError() to be a no-op if Platform::current()
is null. This can happen because MessageLoop destruction may invoke
the callback after Blink shutdown.

TBRing since this precise change was recommended by haraken@ and I'd
like to make sure the fix lands ASAP.

BUG=650637
TBR=haraken@chromium.org

Committed: https://crrev.com/5d0e3432a52018d8134b5c7173a8b649a54ab90d
Cr-Commit-Position: refs/heads/master@{#427190}

[Ken Rockot(use gerrit already), 2016-10-24 20:23:53]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-24 20:24:45]

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

[Ken Rockot(use gerrit already), 2016-10-24 20:42:14]

Description was changed from

==========
Permissions: Avoid poking at the heap after Blink shutdown

Changes serviceConnectionError() to be a no-op if Platform::current()
is null. This can happen because MessageLoop destruction may invoke
the callback after Blink shutdown.

BUG=650637
R=haraken@chromium.org
==========

to

==========
Permissions: Avoid poking at the heap after Blink shutdown

Changes serviceConnectionError() to be a no-op if Platform::current()
is null. This can happen because MessageLoop destruction may invoke
the callback after Blink shutdown.

TBRing since this precise change was recommended by haraken@ and I'd
like to make sure the fix lands ASAP.

BUG=650637
TBR=haraken@chromium.org
==========

[Ken Rockot(use gerrit already), 2016-10-24 20:42:21]

The CQ bit was unchecked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-10-24 20:42:27]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2016-10-24 20:43:51]

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-10-25 00:39:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-25 00:43:27]

Description was changed from

==========
Permissions: Avoid poking at the heap after Blink shutdown

Changes serviceConnectionError() to be a no-op if Platform::current()
is null. This can happen because MessageLoop destruction may invoke
the callback after Blink shutdown.

TBRing since this precise change was recommended by haraken@ and I'd
like to make sure the fix lands ASAP.

BUG=650637
TBR=haraken@chromium.org
==========

to

==========
Permissions: Avoid poking at the heap after Blink shutdown

Changes serviceConnectionError() to be a no-op if Platform::current()
is null. This can happen because MessageLoop destruction may invoke
the callback after Blink shutdown.

TBRing since this precise change was recommended by haraken@ and I'd
like to make sure the fix lands ASAP.

BUG=650637
TBR=haraken@chromium.org

Committed: https://crrev.com/5d0e3432a52018d8134b5c7173a8b649a54ab90d
Cr-Commit-Position: refs/heads/master@{#427190}
==========

[commit-bot: I haz the power, 2016-10-25 00:43:28]

Patchset 1 (id:??) landed as
https://crrev.com/5d0e3432a52018d8134b5c7173a8b649a54ab90d
Cr-Commit-Position: refs/heads/master@{#427190}

[haraken, 2016-10-25 05:14:21]

LGTM