ServiceWorker: Keep info as a member of ServiceWorkerProviderHost

ServiceWorker: Keep info as a member of ServiceWorkerProviderHost

This patch is to use SWProviderHostInfo as an argument of ctor of
SWProviderHost. SWProvider directly keeps the passed SWProviderHostInfo as a
member.

This patch split off from https://crrev.com/2653493009.

BUG=N/A

Review-Url: https://codereview.chromium.org/2891903002
Cr-Commit-Position: refs/heads/master@{#473166}
Committed: https://chromium.googlesource.com/chromium/src/+/b8f57b211d1293781ceac7cc9650983b668cf917

[shimazu, 2017-05-18 08:47:43]

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

[commit-bot: I haz the power, 2017-05-18 08:48:15]

Dry run: CQ is trying da patch.

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

[shimazu, 2017-05-18 09:38:41]

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

[commit-bot: I haz the power, 2017-05-18 09:39:07]

Dry run: CQ is trying da patch.

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

[shimazu, 2017-05-18 10:32:30]

Description was changed from

==========
ServiceWorker: Keep info as a member of ServiceWorkerProviderHost

This patch is to use SWProviderHostInfo as an argument of ctor of
SWProviderHost. SWProvider directly keeps the passed SWProviderHostInfo as a
member.

BUG=N/A
==========

to

==========
ServiceWorker: Keep info as a member of ServiceWorkerProviderHost

This patch is to use SWProviderHostInfo as an argument of ctor of
SWProviderHost. SWProvider directly keeps the passed SWProviderHostInfo as a
member.

This patch split off from https://crrev.com/2653493009.

BUG=N/A
==========

[shimazu, 2017-05-18 10:32:51]

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

[shimazu, 2017-05-18 10:32:51]

ptal, thanks!

[commit-bot: I haz the power, 2017-05-18 13:46:15]

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

[commit-bot: I haz the power, 2017-05-18 13:46:16]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[falken, 2017-05-19 06:42:07]

nice, lgtm

[shimazu, 2017-05-19 08:32:01]

The CQ bit was checked by shimazu@chromium.org

[commit-bot: I haz the power, 2017-05-19 08:32:35]

CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-05-19 08:40:14]

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

[commit-bot: I haz the power, 2017-05-19 08:40:15]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[shimazu, 2017-05-19 08:58:18]

The CQ bit was checked by shimazu@chromium.org

[shimazu, 2017-05-19 08:58: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/2891903002/#ps40001
(title: "Rebased")

[commit-bot: I haz the power, 2017-05-19 08:58:40]

CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-05-19 12:19:23]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1495184298769770,
"parent_rev": "d892b4a534880c5572516e581d3315e1ac103a1f", "commit_rev":
"b8f57b211d1293781ceac7cc9650983b668cf917"}

[commit-bot: I haz the power, 2017-05-19 12:19:33]

Description was changed from

==========
ServiceWorker: Keep info as a member of ServiceWorkerProviderHost

This patch is to use SWProviderHostInfo as an argument of ctor of
SWProviderHost. SWProvider directly keeps the passed SWProviderHostInfo as a
member.

This patch split off from https://crrev.com/2653493009.

BUG=N/A
==========

to

==========
ServiceWorker: Keep info as a member of ServiceWorkerProviderHost

This patch is to use SWProviderHostInfo as an argument of ctor of
SWProviderHost. SWProvider directly keeps the passed SWProviderHostInfo as a
member.

This patch split off from https://crrev.com/2653493009.

BUG=N/A

Review-Url: https://codereview.chromium.org/2891903002
Cr-Commit-Position: refs/heads/master@{#473166}
Committed:
https://chromium.googlesource.com/chromium/src/+/b8f57b211d1293781ceac7cc9650...
==========

[commit-bot: I haz the power, 2017-05-19 12:19:34]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/b8f57b211d1293781ceac7cc9650...

[shimazu, 2017-05-24 02:28:42]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2902823003/ by shimazu@chromium.org.

The reason for reverting is: This seems causing crashes.
See also https://crbug.com/725416..