1608302 - Manage multiple content processes in GeckoProcessManager

Status: RESOLVED FIXED

(GeckoView Graveyard :: Sandboxing, enhancement, P1)

Description

[(No longer employed by Mozilla) Aaron Klotz]

Once the GeckoProcessManager interface supports multiple processes, then we need to change its implementation to actually do so.

Comment 1

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Bug 1608302: Part 1 - Modify service definitions to support multiple content processes; r=#geckoview-reviewers

For testing purposes, we'll only support 3 at the moment.

Note that this does not materially affect our test builds, as e10s-multi is
still govered by the dom.ipc.processCount Gecko pref.

Depends on D65636

Comment 2

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Bug 1608302: Part 2 - Modify GeckoProcessManager to support multiple content processes; r=snorp

Depends on D65639

Comment 3

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Bug 1608302: Part 3 - Update the content process's service name in ContentDelgateTest; r=#geckoview-reviewers

We'll want to make some changes to this test when we enable e10s-multi by
default, but for now we just need to update the name of the single content
process to reflect the naming changes that were done in part 1 of this
patch series.

Depends on D65640

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e75565cd4103
Part 1 - Modify service definitions to support multiple content processes; r=geckoview-reviewers,snorp
https://hg.mozilla.org/integration/autoland/rev/81b75065d43d
Part 2 - Modify GeckoProcessManager to support multiple content processes; r=geckoview-reviewers,snorp
https://hg.mozilla.org/integration/autoland/rev/5a78c81ddaa7
Part 3 - Update the content process's service name in ContentDelgateTest; r=geckoview-reviewers,snorp

Comment 5

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/64e293f463e6
Backed out 6 changesets (bug 1608302, bug 1608301) for linting failure on GeckoProcessManager.java.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c503f5087410
Part 1 - Modify service definitions to support multiple content processes; r=geckoview-reviewers,snorp
https://hg.mozilla.org/integration/autoland/rev/fb19701c39d5
Part 2 - Modify GeckoProcessManager to support multiple content processes; r=geckoview-reviewers,snorp
https://hg.mozilla.org/integration/autoland/rev/714f2025e361
Part 3 - Update the content process's service name in ContentDelgateTest; r=geckoview-reviewers,snorp

Comment 7

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/c503f5087410
https://hg.mozilla.org/mozilla-central/rev/fb19701c39d5
https://hg.mozilla.org/mozilla-central/rev/714f2025e361

Comment 8

[Chris Peterson [:cpeterson]]

Moving content process management bugs to the new GeckoView::Sandboxing component.