ServiceWorker: Ensure that ServiceWorkerDispatcher always adopts passed handle references (2)

ServiceWorker: Ensure that ServiceWorkerDispatcher always adopts passed handle references (2)

Some message handlers on ServiceWorkerDispatcher receive registration/worker
handle references sent from the browser process. Those should surely be
adopted by the dispatcher but not in some early-return cases. This means that
handles in the browser process are leaked until renderer shutdown.

This CL ensures that the dispatcher always adopts passed handle references at
the beginning of message handlers.

Previous CL: https://codereview.chromium.org/1454963003

BUG=557655
Committed: https://crrev.com/7078fc1b5f057ee0f7a4fee34718830eb5e1204a
Cr-Commit-Position: refs/heads/master@{#361593}

[nhiroki, 2015-11-25 04:06:05]

Patchset #1 (id:1) has been deleted

[nhiroki, 2015-11-25 04:06:15]

Patchset #1 (id:20001) has been deleted

[nhiroki, 2015-11-25 04:15:26]

Patchset #1 (id:40001) has been deleted

[nhiroki, 2015-11-25 04:15:40]

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

[nhiroki, 2015-11-25 04:15:41]

ptal, thanks!

[falken, 2015-11-25 04:28:55]

LGTM

[nhiroki, 2015-11-25 05:04:19]

The CQ bit was checked by nhiroki@chromium.org

[nhiroki, 2015-11-25 05:04:19]

The patchset sent to the CQ was uploaded after l-g-t-m from falken@chromium.org
Link to the patchset: https://codereview.chromium.org/1469123003/#ps80001
(title: "fix test")

[commit-bot: I haz the power, 2015-11-25 05:04:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1469123003/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1469123003/80001

[commit-bot: I haz the power, 2015-11-25 06:37:53]

Committed patchset #2 (id:80001)

[commit-bot: I haz the power, 2015-11-25 06:38:47]

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