mirror of
https://github.com/HeyPuter/firefox.git
synced 2026-11-13 20:14:57 -10:00
Yao Xiao
94b9118110
Automatic update from web-platform-tests
[shared storage] Use separate mojom channels for worklet creation (addModule) and for other operations (selectURL/run)
What: Currently, The mojom::SharedStorageDocumentService channel
is used to create the worklet (i.e. handle addModule), and is also
used to route future worklet operations to SharedStorageWorkletHost.
This CL decouples the two things. Note that the new channel is
still associated with mojom::SharedStorageDocumentService (which is
associated with the default navigation mojom channel), so that
worklet operations before navigating away can be handled reliably.
We also combine the handling of "creating worklet" and "addModule"
because they always occur together. By doing this, we can remove some
assertions on addModule status.
Also, move more permission checks to the renderer, and
ReportbadMessage at mojom boundary: 1) addModule can only be called
once, 2) addModule must be called before selectURL()/run(),
3) selectURL()/run() cannot be called if a previous call didn't
have {keepAlive: true}
Why: This prepares for a potential future change to allow multiple
worklets per Document. Nevertheless, a decoupled architecture is
generally better.
Bug: 1218540
Change-Id: Idc2b5df326ba75806f293d881fb9fb00fe80601e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4947893
Reviewed-by: Cammie Smith Barnes <[email protected]>
Commit-Queue: Yao Xiao <[email protected]>
Reviewed-by: Daniel Cheng <[email protected]>
Reviewed-by: Avi Drissman <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1212380}
--
wpt-commits: d093caa9cd12a8a1b9d1e1fa78e1d6b4aaa621d2
wpt-pr: 42590
36 lines
1.1 KiB
HTML
36 lines
1.1 KiB
HTML
<!doctype html>
|
|
<script src="/resources/testharness.js"></script>
|
|
<script src="/resources/testharnessreport.js"></script>
|
|
<script src="/shared-storage/resources/util.js"></script>
|
|
|
|
<body>
|
|
<script>
|
|
'use strict';
|
|
|
|
promise_test(async t => {
|
|
await addModuleOnce("/shared-storage/resources/simple-module.js");
|
|
return sharedStorage.run("operation1", {keepAlive: true});
|
|
}, 'run()');
|
|
|
|
promise_test(async t => {
|
|
await addModuleOnce("/shared-storage/resources/simple-module.js");
|
|
return sharedStorage.run("operation1",
|
|
{data: {'custom-key': 'custom-value'},
|
|
keepAlive: true});
|
|
}, 'run() with custom data');
|
|
|
|
promise_test(async t => {
|
|
try {
|
|
await addModuleOnce("/shared-storage/resources/simple-module.js");
|
|
await sharedStorage.run("operation1", {data: window});
|
|
} catch (e) {
|
|
assert_equals(e.name, 'DataCloneError');
|
|
assert_equals(e.message, 'Failed to execute \'run\' on \'SharedStorage\': #<Window> could not be cloned.');
|
|
return;
|
|
}
|
|
assert_unreached("did not reject");
|
|
}, 'run() with unsupported serializable type');
|
|
|
|
</script>
|
|
</body>
|