1850589 - Fix test_structuredCloneAndExposed.html on Android Fission, with isolateNothing and isolateHighValue

Status: RESOLVED FIXED

(GeckoView Graveyard :: Sandboxing, task)

Description

[[:owlish] 🦉 PST]

STR:

• Run dom/tests/mochitest/whatwg/test_structuredCloneAndExposed.html on any Android build with Fission enabled and fission.webContentIsolationStrategy set to 0 or 2
• The test will fail with assertion TEST-UNEXPECTED-FAIL | dom/tests/mochitest/whatwg/test_structuredCloneAndExposed.html | Deserialization for ImageBitmap in SharedWorkerGlobalScope should succeed - got false, expected true

Logcat for a failure
Logcat for a success

From the logs, it looks like in the success case only one process is started by ActivityManager for the tab, whereas in the failure case - two.

Comment 1

[Nika Layzell [:nika] (ni? for response)]

Attachment: Bug 1850589 - Migrate worker process selection to use ProcessIsolation as a backend, r=smaug!,asuth!

This new backend shares more logic in common with the existing logic in
ProcessIsolation for navigations. This means that it will have support
for precursor principals and alternative process isolation behaviors
like IsolateHighValue on Android.

The logic around custom protocols like web+ and ext+ URIs has been
removed in the new patch, as these will never be used as the principals
for globals. The handling in E10SUtils was just for handling predicting
remote types for URIs before the load has started, which is not relevant
for Workers.

In a future patch we may want to also switch over the prediction logic
which still uses E10SUtils to also use the ProcessIsolation backend,
however it is more different due to the need to handle arbitrary input
URLs, rather than result principals, so the benefit will be smaller.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0953fbf7b4a3
Migrate worker process selection to use ProcessIsolation as a backend, r=smaug,asuth

Comment 3

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/0953fbf7b4a3