1622944 - Make ContentDelegateTest#crashContentMultipleSessions compatible with e10s-multi

Status: RESOLVED FIXED

(GeckoView Graveyard :: Sandboxing, task, P1)

Description

[(No longer employed by Mozilla) Aaron Klotz]

We just need to change this so that we ensure that we get two sessions residing within the same content process.

Comment 1

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Bug 1622944: Update xMultipleSessions junit tests to be sensitive to dom.ipc.processCount; r=#geckoview-reviewers

Since the current e10s process allocation scheme is essentially round-robin, to
get 2 sessions loaded into each process, we need to allocate dom.ipc.processCount * 2
sessions.

Once we've done that, we select the second session loaded into tab0 alongside
mainSession and hook that one into the test.

The previous scheme of waiting on each GeckoResult independently causes
deadlocks if the results are resolved in a different order, so I replaced those
two waits with a GeckoView.allOf result that will Just Work (TM).

Note that this scheme will work both in single-e10s and multi-e10s
configurations.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/929422dd5bc6
Update xMultipleSessions junit tests to be sensitive to dom.ipc.processCount; r=geckoview-reviewers,agi

Comment 3

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/929422dd5bc6

Comment 4

[Chris Peterson [:cpeterson]]

Moving some e10s bugs to the new GeckoView::Sandboxing component.