Raise the bar for SharedArrayBuffer via postMessage() #4734

annevk · 2019-06-25T15:14:57Z

This depends on the work to add Cross-Origin-Opener-Policy and Cross-Origin-Embedder-Policy. It specifies how postMessage() is affected when both those headers are set for the agent clusters they impact.

Closes #4732.

At least two implementers are interested (and none opposed):
- Firefox
- Google
Tests are written and can be reviewed and commented upon at:
Implementation bugs are filed:
- Chrome: …
- Firefox: https://bugzilla.mozilla.org/show_bug.cgi?id=1477743
- Safari: …

(See WHATWG Working Mode: Changes for more details.)

/browsers.html ( diff )
/origin.html ( diff )
/structured-data.html ( diff )
/webappapis.html ( diff )
/workers.html ( diff )

source

For whatwg/html#4734.

The original test could not work. This should be able to work and also adds COOP/COEP headers for future proofing. No browser passes still. For whatwg/html#4734.

For whatwg/html#4734.

annevk · 2019-07-19T14:48:38Z

I've updated OP with a number of tests of which some are landed. I made the decision to land test changes as this is a large undertaking that has multi-implementer buy-in and landing it all at once would it a lot more likely we'd miss important things. The changes should minimally disrupt existing coverage. I'll try to continue to keep OP up-to-date.

…ffer success case revamp, a=testonly Automatic update from web-platform-tests HTML: BroadcastChannel and SharedArrayBuffer success case revamp The original test could not work. This should be able to work and also adds COOP/COEP headers for future proofing. No browser passes still. For whatwg/html#4734. -- wpt-commits: dedb45f5f20e0ac884ac59634cceb8659cdcff7d wpt-pr: 17760

For whatwg/html#4734.

source

…djustments, a=testonly Automatic update from web-platform-tests HTML: SharedArrayBuffer infrastructure adjustments For whatwg/html#4734. -- wpt-commits: a140f239fa9dc4fc3db593b664064b65340e7d4b wpt-pr: 17761 --HG-- rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/broadcastchannel-iframe.html.headers => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/resources/echo-iframe.html.headers rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/broadcastchannel-iframe.html.headers => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/resources/echo-worker.js.headers rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/broadcastchannel-success-and-failure.html => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/broadcastchannel-success-and-failure.https.html rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/identity-not-preserved.html => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/identity-not-preserved.https.html rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/no-transferring.html => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/no-transferring.https.html rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/broadcastchannel-iframe.html.headers => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/broadcastchannel-sharedworker.js.headers rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/broadcastchannel-iframe.html.headers => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/broadcastchannel-worker.js.headers rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/broadcastchannel-iframe.html.headers => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/incrementer-worker-with-channel.js.headers rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/broadcastchannel-iframe.html.headers => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/incrementer-worker.js.headers rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/broadcastchannel-iframe.html.headers => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/nested-worker-success.js.headers rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/broadcastchannel-iframe.html.headers => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/serviceworker-failure.js.headers rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/broadcastchannel-iframe.html.headers => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/sharedworker-failure.js.headers rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/serialization-via-history.html => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/serialization-via-history.https.html rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-domain-success.sub.html => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-domain-success.https.sub.html rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-iframe-messagechannel-success.html => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-iframe-messagechannel-success.https.html rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-messagechannel-success.html => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-messagechannel-success.https.html rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-sharedworker-failure.html => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-sharedworker-failure.https.html rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-similar-but-cross-origin-success.sub.html => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-similar-but-cross-origin-success.https.sub.html rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-simple-success.html => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/window-simple-success.https.html

…er throws without COOP+COEP, a=testonly Automatic update from web-platform-tests HTML: ensure serializing SharedArrayBuffer throws without COOP+COEP For whatwg/html#4734. -- wpt-commits: aecedb53e28abad5432227e293dd016cfc46d4b7 wpt-pr: 17802

…djustments, a=testonly Automatic update from web-platform-tests HTML: SharedArrayBuffer infrastructure adjustments For whatwg/html#4734. -- wpt-commits: a140f239fa9dc4fc3db593b664064b65340e7d4b wpt-pr: 17761

…er throws without COOP+COEP, a=testonly Automatic update from web-platform-tests HTML: ensure serializing SharedArrayBuffer throws without COOP+COEP For whatwg/html#4734. -- wpt-commits: aecedb53e28abad5432227e293dd016cfc46d4b7 wpt-pr: 17802

A top-level navigation response with Cross-Origin-Opener-Policy set to same-origin and Cross-Origin-Embedder-Policy set to require-corp will create a cross-origin isolated browsing context group. And all agent clusters therein will be cross-origin isolated as well (shared and service workers can still not be, as they sit on the side). This change also: * Gates SharedArrayBuffer exposure behind that primitive for web compatibility reasons. * Gates SharedArrayBuffer sharing behind that primitive. * Exposes it through self.crossOriginIsolated. * Makes document.domain return before it mutates the origin. * Makes agent clusters keyed on origin. Tests: * web-platform-tests/wpt#17719 * web-platform-tests/wpt#17760 * web-platform-tests/wpt#17761 * web-platform-tests/wpt#17802 * web-platform-tests/wpt#17909 * web-platform-tests/wpt#18543 * web-platform-tests/wpt#20116 * web-platform-tests/wpt#22358 Closes #4732. Closes #5122. Closes #5444. Follow-up: #5435.

annevk · 2020-06-29T15:43:06Z

I squashed and then rebased and there were no issues. Somewhat strange. I'll work on some fixups now.

domenic

LGTM with nits, hooray

source

annevk · 2020-07-08T08:52:56Z

Oh wait, given the wording in tc39/ecma262#1903 I guess it's solely up to the host, not the implementation. So this is all good.

…ffer success case revamp, a=testonly Automatic update from web-platform-tests HTML: BroadcastChannel and SharedArrayBuffer success case revamp The original test could not work. This should be able to work and also adds COOP/COEP headers for future proofing. No browser passes still. For whatwg/html#4734. -- wpt-commits: dedb45f5f20e0ac884ac59634cceb8659cdcff7d wpt-pr: 17760

…djustments, a=testonly Automatic update from web-platform-tests HTML: SharedArrayBuffer infrastructure adjustments For whatwg/html#4734. -- wpt-commits: a140f239fa9dc4fc3db593b664064b65340e7d4b wpt-pr: 17761

…er throws without COOP+COEP, a=testonly Automatic update from web-platform-tests HTML: ensure serializing SharedArrayBuffer throws without COOP+COEP For whatwg/html#4734. -- wpt-commits: aecedb53e28abad5432227e293dd016cfc46d4b7 wpt-pr: 17802

Automatic update from web-platform-tests HTML: COEP and ImageBitmap Helps with whatwg/html#4734 and #18354. -- wpt-commits: 53c7a93cc4ba3a5fa1fe3f3358eeb9ad2ad2f0e9 wpt-pr: 18543

This test was used to verify that postMessaging to a same-origin-domain (but not same-origin) iframe allows them to see each others' modification. However, we now only allow same-origin to postMessage SharedArrayBuffer. And, that means same-origin-domain are not allowed to do that. Therefore, this test is rewrited to verify that postMessage SharedArrayBuffer from a same-origin-domain iframe would result the receiver (parent) to get a message error event. Github issue: whatwg/html#4734 Differential Revision: https://phabricator.services.mozilla.com/D70348

…; r=annevk Changes for using messagechannel to share the SharedArrayBuffer. Github issue: whatwg/html#4734 Differential Revision: https://phabricator.services.mozilla.com/D70349

….https.sub.html; r=annevk We have window-domain-failure.https.sub.html so we don't need this test. Github issue: whatwg/html#4734 Differential Revision: https://phabricator.services.mozilla.com/D70350

annevk added do not merge yet Pull request must not be merged per rationale in comment needs tests Moving the issue forward requires someone to write tests topic: serialize and transfer labels Jun 25, 2019

annevk commented Jun 26, 2019

View reviewed changes

source Outdated Show resolved Hide resolved

domenic reviewed Jul 2, 2019

View reviewed changes

source Outdated Show resolved Hide resolved

source Outdated Show resolved Hide resolved

source Outdated Show resolved Hide resolved

annevk mentioned this pull request Jul 9, 2019

Update SharedArrayBuffer tests to account for COOP/COEP web-platform-tests/wpt#17690

Closed

annevk commented Jul 11, 2019

View reviewed changes

source Outdated Show resolved Hide resolved

annevk mentioned this pull request Jul 11, 2019

COEP and cachestorage #4764

Closed

domenic reviewed Jul 11, 2019

View reviewed changes

source Outdated Show resolved Hide resolved

annevk mentioned this pull request Jul 11, 2019

COEP and CORP opaque responses #4767

Closed

annevk added a commit to web-platform-tests/wpt that referenced this pull request Jul 12, 2019

HTML: ensure serializing SharedArrayBuffer throws without COOP+COEP

69de2b9

For whatwg/html#4734.

annevk mentioned this pull request Jul 12, 2019

HTML: ensure serializing SharedArrayBuffer throws without COOP+COEP web-platform-tests/wpt#17802

Merged

annevk added needs tests Moving the issue forward requires someone to write tests and removed needs tests Moving the issue forward requires someone to write tests labels Jul 19, 2019

annevk added a commit to web-platform-tests/wpt that referenced this pull request Jul 19, 2019

HTML: SharedArrayBuffer infrastructure adjustments

9559f08

For whatwg/html#4734.

annevk added a commit to web-platform-tests/wpt that referenced this pull request Jul 19, 2019

HTML: SharedArrayBuffer infrastructure adjustments

581fb3e

For whatwg/html#4734.

annevk added a commit to web-platform-tests/wpt that referenced this pull request Aug 6, 2019

HTML: SharedArrayBuffer infrastructure adjustments

a140f23

For whatwg/html#4734.

annevk added a commit to web-platform-tests/wpt that referenced this pull request Aug 6, 2019

HTML: ensure serializing SharedArrayBuffer throws without COOP+COEP

aecedb5

For whatwg/html#4734.

annevk mentioned this pull request Aug 9, 2019

Remaining COOP and COEP testing work web-platform-tests/wpt#18354

Open

16 tasks

annevk commented Aug 9, 2019

View reviewed changes

source Outdated Show resolved Hide resolved

This comment has been minimized.

Sign in to view

annevk force-pushed the annevk/postmessage-sab branch from b465512 to c772611 Compare August 14, 2019 11:54

annevk force-pushed the annevk/postmessage-sab branch from a27a45d to 50ebcb4 Compare June 29, 2020 15:42

address feedback

4e5a00c

annevk removed the do not merge yet Pull request must not be merged per rationale in comment label Jun 29, 2020

domenic approved these changes Jul 6, 2020

View reviewed changes

source Outdated Show resolved Hide resolved

source Show resolved Hide resolved

annevk and others added 2 commits July 7, 2020 12:36

getter steps

fd3f73a

More precise SAB deletion

522b16e

annevk mentioned this pull request Jul 8, 2020

SharedWorker is racy #5362

Open

nit

f1e41e5

annevk merged commit c9d8983 into master Jul 8, 2020

annevk deleted the annevk/postmessage-sab branch July 8, 2020 09:05

github-actions bot mentioned this pull request Apr 26, 2021

In JS docs, use Note: consistently at the start of notes mdn/content#4504

Merged

github-actions bot mentioned this pull request Aug 26, 2021

Move globals (ex WindowOrWorkerGlobalScope) under API/* mdn/content#8351

Merged

github-actions bot mentioned this pull request Jan 12, 2022

Global_Objects/SharedArrayBuffer 以下の記事を更新 mdn/translated-content#3647

Merged

github-actions bot mentioned this pull request May 9, 2022

Initial translation for missing SharedArrayBuffer pages mdn/translated-content#5516

Merged

github-actions bot mentioned this pull request Sep 18, 2022

Merge "SAB planned changes" into the main page mdn/content#20863

Merged

github-actions bot mentioned this pull request Jan 3, 2023

SharedArrayBuffer の記事を更新 mdn/translated-content#10881

Merged

github-actions bot mentioned this pull request Feb 21, 2023

JavaScript: Tag removal/status - globals s*r* mdn/content#24660

Merged

github-actions bot mentioned this pull request Aug 3, 2023

ja: Lint front matter keys for "conflicting" mdn/translated-content#14886

Closed

github-actions bot mentioned this pull request Aug 20, 2023

Markdownlint auto-cleanup for ja mdn/translated-content#15282

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Raise the bar for SharedArrayBuffer via postMessage() #4734

Raise the bar for SharedArrayBuffer via postMessage() #4734

annevk commented Jun 25, 2019 •

edited by pr-preview bot

Loading

annevk commented Jul 19, 2019

This comment has been minimized.

annevk commented Jun 29, 2020

domenic left a comment

annevk commented Jul 8, 2020

Raise the bar for SharedArrayBuffer via postMessage() #4734

Raise the bar for SharedArrayBuffer via postMessage() #4734

Conversation

annevk commented Jun 25, 2019 • edited by pr-preview bot Loading

annevk commented Jul 19, 2019

This comment has been minimized.

annevk commented Jun 29, 2020

domenic left a comment

Choose a reason for hiding this comment

annevk commented Jul 8, 2020

annevk commented Jun 25, 2019 •

edited by pr-preview bot

Loading