PlzNavigate: ensure we don't create extra-processes for ServiceWorkers

PlzNavigate: ensure we don't create extra-processes for ServiceWorkers

This CL adds a function to SiteInstanceImpl that finds a
SiteInstance that can be used to display a site based on a
BrowserContext and uses in ServiceWorkerProcessManager when no
RenderProcessHost is already in use for a particular pattern. In order
for this function to work, the SiteInstance now records which site it
was used to display. We also keep track of BrowsingInstances to be able
to iterate over all BrowsingInstances in order to find a SiteInstance.

BUG=705318
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

[clamy, 2017-04-07 13:55:50]

Description was changed from

==========
PlzNavigate: ensure we don't create extra-processes for ServiceWorkers

This CL adds a function to SiteInstanceImpl that finds a
SiteInstance that can be used to display a site based on a
BrowserContext and uses in ServiceWorkerProcessManager when no
RenderProcessHost is already in use for a particular pattern. In order
for this function to work, the SiteInstance now records which site it
was used to display. We also keep track of BrowsingInstances to be able
to iterate over all BrowsingInstances in order to find a SiteInstance.

BUG=705318
==========

to

==========
PlzNavigate: ensure we don't create extra-processes for ServiceWorkers

This CL adds a function to SiteInstanceImpl that finds a
SiteInstance that can be used to display a site based on a
BrowserContext and uses in ServiceWorkerProcessManager when no
RenderProcessHost is already in use for a particular pattern. In order
for this function to work, the SiteInstance now records which site it
was used to display. We also keep track of BrowsingInstances to be able
to iterate over all BrowsingInstances in order to find a SiteInstance.

BUG=705318
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[jam1, 2017-04-07 16:38:58]

The CQ bit was checked by jam@google.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-07 16:39:32]

Dry run: 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-04-07 17:48:38]

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

[commit-bot: I haz the power, 2017-04-07 17:48:39]

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

[Charlie Reis, 2017-05-03 23:28:49]

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

[Charlie Reis, 2017-05-03 23:28:49]

Just a note: I think we're probably abandoning this in favor of the approach on
https://codereview.chromium.org/2861433002/.

[jam, 2017-05-10 23:34:14]

Description was changed from

==========
PlzNavigate: ensure we don't create extra-processes for ServiceWorkers

This CL adds a function to SiteInstanceImpl that finds a
SiteInstance that can be used to display a site based on a
BrowserContext and uses in ServiceWorkerProcessManager when no
RenderProcessHost is already in use for a particular pattern. In order
for this function to work, the SiteInstance now records which site it
was used to display. We also keep track of BrowsingInstances to be able
to iterate over all BrowsingInstances in order to find a SiteInstance.

BUG=705318
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
PlzNavigate: ensure we don't create extra-processes for ServiceWorkers

This CL adds a function to SiteInstanceImpl that finds a
SiteInstance that can be used to display a site based on a
BrowserContext and uses in ServiceWorkerProcessManager when no
RenderProcessHost is already in use for a particular pattern. In order
for this function to work, the SiteInstance now records which site it
was used to display. We also keep track of BrowsingInstances to be able
to iterate over all BrowsingInstances in order to find a SiteInstance.

BUG=705318
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[jam, 2017-05-10 23:34:14]

jam@chromium.org changed reviewers:
- jam@google.com