PlzNavigate: use a SiteInstance to intialize the ServiceWorkerNavigationHandle

PlzNavigate: use a SiteInstance to intialize the ServiceWorkerNavigationHandle

With this CL the StoragePartition used to get acces to the
ServiceWorkerContextWrapper is accessed thanks to a SiteInstance instead of the
navigation url. This fixes a bug in which WebViews navigating would initialize
the ServiceWorkerNavigationHandle with the wrong ServiceWorkerContextWrapper,
leading to the precreated ServiceWorkerProviderHost being stored in a wrong
ServiceWorkerContextCore. When the renderer latter attempted to retrieve it, it
would get killed due to the ServiceWorkerProviderHost not being found. This
fixes ~90 WebView browser tests with PlzNavigate enabled.

BUG=504347
Committed: https://crrev.com/7ba89802b549276c56d414fd8418fcf887e73342
Cr-Commit-Position: refs/heads/master@{#361673}

[clamy, 2015-11-24 15:17:10]

Description was changed from

==========
PlzNavigate: use a SiteInstance to intialize the ServiceWorkerNavigationHandle

With this CL the StoragePartition used to get acces to the
ServiceWorkerContextWrapper is accessed thanks to a SiteInstance instead of the
navigation url. This fixes a bug in which WebViews navigating would initialize
the ServiceWorkerNavigationHandle with the wrong ServiceWorkerContextWrapper,
leading to the precreated ServiceWorkerProviderHost being stored in a wrong
ServiceWorkerContextCore. When the renderer latter attempted to retrieve it, it
would get killed due to the ServiceWorkerProviderHost not being found. This
fixes ~90 WebView browser tests with PlzNavigate enabled.

BUG=504347
==========

to

==========
PlzNavigate: use a SiteInstance to intialize the ServiceWorkerNavigationHandle

With this CL the StoragePartition used to get acces to the
ServiceWorkerContextWrapper is accessed thanks to a SiteInstance instead of the
navigation url. This fixes a bug in which WebViews navigating would initialize
the ServiceWorkerNavigationHandle with the wrong ServiceWorkerContextWrapper,
leading to the precreated ServiceWorkerProviderHost being stored in a wrong
ServiceWorkerContextCore. When the renderer latter attempted to retrieve it, it
would get killed due to the ServiceWorkerProviderHost not being found. This
fixes ~90 WebView browser tests with PlzNavigate enabled.

BUG=504347
==========

[clamy, 2015-11-24 15:17:10]

clamy@chromium.org changed reviewers:
+ nasko@chromium.org

[clamy, 2015-11-24 15:17:22]

@nasko: PTAL

[nasko, 2015-11-25 14:51:44]

nasko@chromium.org changed reviewers:
+ davidben@chromium.org

[nasko, 2015-11-25 14:51:44]

Adding davidben@ as FYI, as he has been thinking about StoragePartitions and
ServiceWorkers and this CL combines both.

[nasko, 2015-11-25 14:51:58]

LGTM

[clamy, 2015-11-25 15:29:40]

The CQ bit was checked by clamy@chromium.org

[clamy, 2015-11-25 15:29:41]

The patchset sent to the CQ was uploaded after l-g-t-m from nasko@chromium.org
Link to the patchset: https://codereview.chromium.org/1473013003/#ps20001
(title: "Not depending on codereview.chromium.org/1467563002 anymore")

[commit-bot: I haz the power, 2015-11-25 15:29:58]

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

[commit-bot: I haz the power, 2015-11-25 16:30:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-25 16:31:21]

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