Save running SW instance info, including its SiteInstance, into the ProcessManager.

Save running SW instance info, including its SiteInstance, into the ProcessManager.

We can use this map to drop all the process references on shutdown, in order to
fix http://crbug.com/368570.

This starts to fix an assumption in the content layer that SiteInstances outlive
their RPHs so that process permissions can be controlled through the
SiteInstance instead of by process id. However, there's still work to do before
that's completely fixed.

This reverts https://codereview.chromium.org/296053013/ and avoids the incorrect
down-cast that caused https://codereview.chromium.org/292903002 to break the
Asan bot.

BUG=368570
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272490

[Jeffrey Yasskin, 2014-05-22 22:13:34]

In writing the ChooseProcess test with a MockRenderProcessHost, I forgot that
part of the implementation down-casts to RenderProcessHostImpl in order to
increment the worker count. Since they're different types, that's an
out-of-bounds write.

This version shrinks the test to only focus on SortProcesses instead of passing
the result through AllocateWorkerProcess.

[kinuko, 2014-05-23 02:51:19]

lgtm

[kinuko, 2014-05-23 02:53:23]

https://codereview.chromium.org/294073009/diff/20001/content/browser/service_...
File content/browser/service_worker/embedded_worker_instance_unittest.cc
(right):

https://codereview.chromium.org/294073009/diff/20001/content/browser/service_...
content/browser/service_worker/embedded_worker_instance_unittest.cc:150:
testing::ElementsAre(3, 2, 1));
Neat, didn't know this notation.

[kinuko, 2014-05-23 02:53:29]

The CQ bit was checked by kinuko@chromium.org

[commit-bot: I haz the power, 2014-05-23 02:55:23]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jyasskin@chromium.org/294073009/20001

[commit-bot: I haz the power, 2014-05-23 11:29:23]

Change committed as 272490