Remove WebPrerenderingSupport::shutdown from blink::shutdown

Remove WebPrerenderingSupport::shutdown from blink::shutdown

WebPrerenderingSupport::initialize/shutdown is called in
PrerenderDispatcher's constructor/destructor. There would be no need to
call WebPrerenderingSupport::shutdown from blink::shutdown.
(It looks strange to call WebPrerenderingSupport::shutdown in blink::shutdown
whereas WebPrerenderingSupport::initialize is not called in blink::initialize.)

The code was added in https://chromium.googlesource.com/chromium/src/+/b5d48114fbb738a58cb58b9023c1e25ea94a9ec0 four years ago.

BUG=593269
Committed: https://crrev.com/3a93d12a5e0b250e9c0e9b3b5d47208454c5e350
Cr-Commit-Position: refs/heads/master@{#380845}

[haraken, 2016-03-09 12:17:28]

haraken@chromium.org changed reviewers:
+ gavinp@chromium.org, sigbjornf@opera.com

[haraken, 2016-03-09 12:17:29]

PTAL

[sof, 2016-03-09 12:34:08]

This is for gavinp@ , but making the shutdown of PrerenderDispatcher and Blink
happen in a guaranteed coordinated matter might be the reason (vaguely reminded
of troubles from issue 414402). But a pure guess.

[sof, 2016-03-09 12:35:38]

On 2016/03/09 12:34:08, sof wrote:
> This is for gavinp@ , but making the shutdown of PrerenderDispatcher and Blink
> happen in a guaranteed coordinated matter might be the reason (vaguely
reminded
> of troubles from issue 414402). But a pure guess.

s/matter/manner/ - need more coffee.

[haraken, 2016-03-10 11:09:36]

gavinp@: Would you take a look at this?

[gavinp, 2016-03-11 19:08:35]

On 2016/03/10 11:09:36, haraken wrote:
> gavinp@: Would you take a look at this?

lgtm. I'm digging deeper into 414402 as it's coming back. Thanks for the tip
sof. I'm thinking this is more just an oversight though.

[haraken, 2016-03-12 02:13:49]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-03-12 02:14:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1771393006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1771393006/20001

[commit-bot: I haz the power, 2016-03-12 02:35:59]

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

[commit-bot: I haz the power, 2016-03-12 02:36:00]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[haraken, 2016-03-12 02:58:02]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-03-12 02:58:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1771393006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1771393006/20001

[commit-bot: I haz the power, 2016-03-12 04:43:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-12 04:45:53]

Description was changed from

==========
Remove WebPrerenderingSupport::shutdown from blink::shutdown

WebPrerenderingSupport::initialize/shutdown is called in
PrerenderDispatcher's constructor/destructor. There would be no need to
call WebPrerenderingSupport::shutdown from blink::shutdown.
(It looks strange to call WebPrerenderingSupport::shutdown in blink::shutdown
whereas WebPrerenderingSupport::initialize is not called in blink::initialize.)

The code was added in
https://chromium.googlesource.com/chromium/src/+/b5d48114fbb738a58cb58b9023c1...
four years ago.

BUG=593269
==========

to

==========
Remove WebPrerenderingSupport::shutdown from blink::shutdown

WebPrerenderingSupport::initialize/shutdown is called in
PrerenderDispatcher's constructor/destructor. There would be no need to
call WebPrerenderingSupport::shutdown from blink::shutdown.
(It looks strange to call WebPrerenderingSupport::shutdown in blink::shutdown
whereas WebPrerenderingSupport::initialize is not called in blink::initialize.)

The code was added in
https://chromium.googlesource.com/chromium/src/+/b5d48114fbb738a58cb58b9023c1...
four years ago.

BUG=593269

Committed: https://crrev.com/3a93d12a5e0b250e9c0e9b3b5d47208454c5e350
Cr-Commit-Position: refs/heads/master@{#380845}
==========

[commit-bot: I haz the power, 2016-03-12 04:45:55]

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