Worker: Early return if termination is requested while worker thread is starting

Worker: Early return if termination is requested while worker thread is starting

WorkerThread may get ready for shutdown if termination is requested while a
debugger task is running. In this case, WorkerThread disposes WorkerGlobalScope
etc, so it should not resume initialization.

BUG=615839
Committed: https://crrev.com/52c1c59812f930cd7056b9d6aa8c14213f2c70a9
Cr-Commit-Position: refs/heads/master@{#401543}

[nhiroki, 2016-06-17 21:12:32]

Description was changed from

==========
Worker: Early return if termination is requested while worker thread is starting

BUG=615839
==========

to

==========
Worker: Early return if termination is requested while worker thread is starting

WorkerThread may get ready for shutdown if termination is requested while a
debugger task is running. In this case, WorkerThread disposes WorkerGlobalScope
etc, so it should not resume initialization.

BUG=615839
==========

[nhiroki, 2016-06-17 21:29:01]

nhiroki@chromium.org changed reviewers:
+ kinuko@chromium.org

[nhiroki, 2016-06-17 21:29:02]

PTAL, thanks!

[kinuko, 2016-06-20 09:16:46]

Hmm, ok. Should we have a test (in a follow-up)? lgtm anyways

[nhiroki, 2016-06-21 17:46:32]

Thank you! I added a test. Can you take another look? :)

[nhiroki, 2016-06-23 03:57:48]

kinuko-san, friendly ping

[kinuko, 2016-06-23 04:05:58]

On 2016/06/23 03:57:48, nhiroki wrote:
> kinuko-san, friendly ping

still lgtm

[nhiroki, 2016-06-23 04:08:20]

On 2016/06/23 04:05:58, kinuko wrote:
> On 2016/06/23 03:57:48, nhiroki wrote:
> > kinuko-san, friendly ping
> 
> still lgtm

Thank you!

[nhiroki, 2016-06-23 04:08:25]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2016-06-23 04:08:43]

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

[commit-bot: I haz the power, 2016-06-23 05:37:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-23 05:39:56]

Description was changed from

==========
Worker: Early return if termination is requested while worker thread is starting

WorkerThread may get ready for shutdown if termination is requested while a
debugger task is running. In this case, WorkerThread disposes WorkerGlobalScope
etc, so it should not resume initialization.

BUG=615839
==========

to

==========
Worker: Early return if termination is requested while worker thread is starting

WorkerThread may get ready for shutdown if termination is requested while a
debugger task is running. In this case, WorkerThread disposes WorkerGlobalScope
etc, so it should not resume initialization.

BUG=615839

Committed: https://crrev.com/52c1c59812f930cd7056b9d6aa8c14213f2c70a9
Cr-Commit-Position: refs/heads/master@{#401543}
==========

[commit-bot: I haz the power, 2016-06-23 05:39:58]

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