ServiceWorker: Stop a worker thread on unregistration

ServiceWorker: Stop a worker thread on unregistration

ServiceWorker is designed to delay stopping the worker thread (30 sec) when all
controlled documents are closed so that the next navigation can save the startup
time.

In the current code, this improvement also applies to the unregistration with no
controllees, but in that case we should immediately stop the thread because the
worker doesn't work anymore.


BUG=392735, 393127
TEST=content_unittests --gtest_filter=ServiceWorker*

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282922

[nhiroki, 2014-07-11 09:40:09]

Is this reasonable change? If so, I'll make a test. Thanks!

[falken, 2014-07-12 05:51:01]

Yep this looks right. Also relevant is
https://github.com/slightlyoff/ServiceWorker/issues/346

[nhiroki, 2014-07-14 04:22:03]

Thanks for your comment. Updated and removed WIP mark. Can you take another
look?

[falken, 2014-07-14 07:27:34]

On 2014/07/14 04:22:03, nhiroki wrote:
> Thanks for your comment. Updated and removed WIP mark. Can you take another
> look?

lgtm

nit: "but in the case" -> "but in that case"

[nhiroki, 2014-07-14 07:32:07]

On 2014/07/14 07:27:34, falken wrote:
> On 2014/07/14 04:22:03, nhiroki wrote:
> > Thanks for your comment. Updated and removed WIP mark. Can you take another
> > look?
> 
> lgtm
> 
> nit: "but in the case" -> "but in that case"

Thanks! Done.

[nhiroki, 2014-07-14 07:33:16]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2014-07-14 07:34:07]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nhiroki@chromium.org/385143002/20001

[commit-bot: I haz the power, 2014-07-14 09:16:00]

Change committed as 282922