Stop calling Platform::shutdown when a PPAPI/utility thread stop

Stop calling Platform::shutdown when a PPAPI/utility thread stop

As discussed in blink-dev@ (https://groups.google.com/a/chromium.org/d/topic/blink-dev/kk4VX0xRB7I/discussion)
and platform-architecture-dev@ (https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/Zc12k91NTFk/discussion),
there is no reason we have to shut down the renderer gracefully.
It's just causing use-after-free bugs and wasting performance.
This CL stops calling Platform::shutdown() when a PPAPI/utility thread stop.

BUG=639244
Committed: https://crrev.com/9953069136019c8fb2489763f925dc291d94c7c2
Cr-Commit-Position: refs/heads/master@{#415297}

[haraken, 2016-08-25 01:41:55]

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

[haraken, 2016-08-25 01:42:16]

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

[haraken, 2016-08-25 01:42:16]

PTAL

[commit-bot: I haz the power, 2016-08-25 01:43:06]

Dry run: 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 04:04:32]

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

[commit-bot: I haz the power, 2016-08-25 04:04:32]

Dry run: This issue passed the CQ dry run.

[haraken, 2016-08-26 00:55:07]

jam@: friendly ping?

[haraken, 2016-08-27 00:36:53]

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

[haraken, 2016-08-27 00:36:53]

Elliott: Would you take a look at this?

[esprehn, 2016-08-27 01:21:59]

lgtm

[haraken, 2016-08-27 01:23:20]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-08-27 01:23: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-27 01:29:40]

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

[commit-bot: I haz the power, 2016-08-27 01:29:42]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[haraken, 2016-08-29 15:29:50]

jam@: Would you take a look as an owner?

[haraken, 2016-08-30 03:32:22]

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

[haraken, 2016-08-30 03:32:22]

jochen@: Would you take a look at this?

[jochen (gone - plz use gerrit), 2016-08-30 11:51:58]

lgtm

[haraken, 2016-08-30 14:14:26]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-08-30 14:14:43]

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-30 15:08:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 15:09:45]

Description was changed from

==========
Stop calling Platform::shutdown when a PPAPI/utility thread stop

As discussed in blink-dev@
(https://groups.google.com/a/chromium.org/d/topic/blink-dev/kk4VX0xRB7I/discus...)
and platform-architecture-dev@
(https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/Zc...),
there is no reason we have to shut down the renderer gracefully.
It's just causing use-after-free bugs and wasting performance.
This CL stops calling Platform::shutdown() when a PPAPI/utility thread stop.

BUG=639244
==========

to

==========
Stop calling Platform::shutdown when a PPAPI/utility thread stop

As discussed in blink-dev@
(https://groups.google.com/a/chromium.org/d/topic/blink-dev/kk4VX0xRB7I/discus...)
and platform-architecture-dev@
(https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/Zc...),
there is no reason we have to shut down the renderer gracefully.
It's just causing use-after-free bugs and wasting performance.
This CL stops calling Platform::shutdown() when a PPAPI/utility thread stop.

BUG=639244

Committed: https://crrev.com/9953069136019c8fb2489763f925dc291d94c7c2
Cr-Commit-Position: refs/heads/master@{#415297}
==========

[commit-bot: I haz the power, 2016-08-30 15:09:46]

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