Remove Platform::current checks from Mojo's error handlers in geolocation

Remove Platform::current checks from Mojo's error handlers in geolocation

The checks were added in https://codereview.chromium.org/2163683004/
but are no longer needed because I already removed blink::shutdown
from the renderer's shutdown sequence.

BUG=628999, 639244
Committed: https://crrev.com/8435863a81be1eee4c28131a341f8a4bce04a7b4
Cr-Commit-Position: refs/heads/master@{#414232}

[haraken, 2016-08-24 07:15:21]

haraken@chromium.org changed reviewers:
+ sammc@chromium.org

[haraken, 2016-08-24 07:15:22]

PTAL

[Sam McNally, 2016-08-24 21:42:07]

lgtm

[haraken, 2016-08-24 22:49:28]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-08-24 22:50:50]

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

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-25 00:56:18]

Description was changed from

==========
Remove Platform::current checks from Mojo's error handlers in geolocation

The checks were added in https://codereview.chromium.org/2163683004/
but are no longer needed because I already removed blink::shutdown
from the renderer's shutdown sequence.

BUG=628999
==========

to

==========
Remove Platform::current checks from Mojo's error handlers in geolocation

The checks were added in https://codereview.chromium.org/2163683004/
but are no longer needed because I already removed blink::shutdown
from the renderer's shutdown sequence.

BUG=628999

Committed: https://crrev.com/8435863a81be1eee4c28131a341f8a4bce04a7b4
Cr-Commit-Position: refs/heads/master@{#414232}
==========

[commit-bot: I haz the power, 2016-08-25 00:56:20]

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

[haraken, 2016-09-05 00:27:24]

Description was changed from

==========
Remove Platform::current checks from Mojo's error handlers in geolocation

The checks were added in https://codereview.chromium.org/2163683004/
but are no longer needed because I already removed blink::shutdown
from the renderer's shutdown sequence.

BUG=628999

Committed: https://crrev.com/8435863a81be1eee4c28131a341f8a4bce04a7b4
Cr-Commit-Position: refs/heads/master@{#414232}
==========

to

==========
Remove Platform::current checks from Mojo's error handlers in geolocation

The checks were added in https://codereview.chromium.org/2163683004/
but are no longer needed because I already removed blink::shutdown
from the renderer's shutdown sequence.

BUG=628999,639244

Committed: https://crrev.com/8435863a81be1eee4c28131a341f8a4bce04a7b4
Cr-Commit-Position: refs/heads/master@{#414232}
==========

[haraken, 2016-09-05 00:28:29]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2312553003/ by haraken@chromium.org.

The reason for reverting is: I'll revert r413430 and its dependent CLs because
r413430 caused issue 642072.

The problem is that we cannot simply remove blink::shutdown because the
following scenario can happen:

1) blink::shutdown is not called. Workers are still running.
2) RenderThreadImpl gets destructed. MessageLoop gets destructed.
3) The workers may access the RenderThreadImpl and MessageLoop.

To fix the problem, we need to call ProcessDied() and forcibly kill the renderer
process at the end of RenderThreadImpl::Shutdown().
.