(WIP) ServiceWorker: Remove unnecessary WorkerThread::terminateAndWait() call

ServiceWorker: Remove unnecessary WorkerThread::terminateAndWait() call

This CL removes WorkerThread::terminateAndWait() call from the dtor of
WebEmbeddedWorkerImpl. WorkerThread should already be terminated or should not
be started yet at the point.

What if the owner of WebEmbeddedWorkerImpl (ie. EmbeddedWorkerDispatcher) is
destroyed before the worker thread is terminated. This is not a matter because
it possibly happened on graceful renderer process shutdown but we already
abondoned the sequence: https://crbug.com/639244

BUG=641846

[nhiroki, 2016-08-29 05:16:00]

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

[commit-bot: I haz the power, 2016-08-29 05:16:25]

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

[nhiroki, 2016-08-29 05:34:24]

Description was changed from

==========
ServiceWorker: Remove unnecessary WorkerThread::terminateAndWait() call

This CL removes WorkerThread::terminateAndWait() call from the dtor of
WebEmbeddedWorkerImpl. WorkerThread should already be terminated or should not
be started yet at the point.

BUG=641846
==========

to

==========
ServiceWorker: Remove unnecessary WorkerThread::terminateAndWait() call

This CL removes WorkerThread::terminateAndWait() call from the dtor of
WebEmbeddedWorkerImpl. WorkerThread should already be terminated or should not
be started yet at the point.

What if the owner of WebEmbeddedWorkerImpl (ie. EmbeddedWorkerDispatcher) is
destroyed before the worker thread is terminated. This is not a matter because
it possibly happened on graceful renderer process shutdown but we already
abondoned the sequence: https://crbug.com/639244

BUG=641846
==========

[nhiroki, 2016-08-29 05:35:25]

Description was changed from

==========
ServiceWorker: Remove unnecessary WorkerThread::terminateAndWait() call

This CL removes WorkerThread::terminateAndWait() call from the dtor of
WebEmbeddedWorkerImpl. WorkerThread should already be terminated or should not
be started yet at the point.

What if the owner of WebEmbeddedWorkerImpl (ie. EmbeddedWorkerDispatcher) is
destroyed before the worker thread is terminated. This is not a matter because
it possibly happened on graceful renderer process shutdown but we already
abondoned the sequence: https://crbug.com/639244

BUG=641846
==========

to

==========
ServiceWorker: Remove unnecessary WorkerThread::terminateAndWait() call

This CL removes WorkerThread::terminateAndWait() call from the dtor of
WebEmbeddedWorkerImpl. WorkerThread should already be terminated or should not
be started yet at the point.

What if the owner of WebEmbeddedWorkerImpl (ie. EmbeddedWorkerDispatcher) is
destroyed before the worker thread is terminated. This is not a matter because
it possibly happened on graceful renderer process shutdown but we already
abondoned the sequence: https://crbug.com/639244

BUG=641846
==========

[commit-bot: I haz the power, 2016-08-29 06:25:27]

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

[commit-bot: I haz the power, 2016-08-29 06:25:28]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)