Add a return when StopWorker is called after service worker is stopped

Add a return when StopWorker is called after service worker is stopped

Currently StopWorker might be crashed when stop is requested after the worker's
context is destoryed. This patch adds a check condition if the worker is
already stopped.

BUG=664014
Committed: https://crrev.com/a32c1633d7f194771bb32c38c9ff100d34e49642
Cr-Commit-Position: refs/heads/master@{#431543}

[shimazu, 2016-11-11 10:23:05]

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

[commit-bot: I haz the power, 2016-11-11 10:23:17]

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

[shimazu, 2016-11-11 10:25:37]

shimazu@chromium.org changed reviewers:
+ falken@chromium.org

[shimazu, 2016-11-11 10:25:37]

ptal

[falken, 2016-11-11 10:27:59]

lgtm

[falken, 2016-11-11 10:28:17]

The CQ bit was unchecked by falken@chromium.org

[falken, 2016-11-11 10:28:25]

The CQ bit was checked by falken@chromium.org

[commit-bot: I haz the power, 2016-11-11 10:28:39]

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

[nhiroki, 2016-11-11 11:23:39]

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

[nhiroki, 2016-11-11 11:24:10]

https://codereview.chromium.org/2498573002/diff/1/content/renderer/service_wo...
File content/renderer/service_worker/embedded_worker_instance_client_impl.cc
(right):

https://codereview.chromium.org/2498573002/diff/1/content/renderer/service_wo...
content/renderer/service_worker/embedded_worker_instance_client_impl.cc:68:
wrapper_ = wrapper.get();
DCHECK(!wrapper_);

https://codereview.chromium.org/2498573002/diff/1/content/renderer/service_wo...
content/renderer/service_worker/embedded_worker_instance_client_impl.cc:92:
temporal_self_(std::unique_ptr<EmbeddedWorkerInstanceClientImpl>(this)) {
How about initializing |wrapper_| with nullptr?

[commit-bot: I haz the power, 2016-11-11 11:52:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-11 11:57:29]

Description was changed from

==========
Add a return when StopWorker is called after service worker is stopped

Currently StopWorker might be crashed when stop is requested after the worker's
context is destoryed. This patch adds a check condition if the worker is
already stopped.

BUG=664014
==========

to

==========
Add a return when StopWorker is called after service worker is stopped

Currently StopWorker might be crashed when stop is requested after the worker's
context is destoryed. This patch adds a check condition if the worker is
already stopped.

BUG=664014

Committed: https://crrev.com/a32c1633d7f194771bb32c38c9ff100d34e49642
Cr-Commit-Position: refs/heads/master@{#431543}
==========

[commit-bot: I haz the power, 2016-11-11 11:57:30]

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