Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[spec] Support custom dataOrigin in createWorklet #206

Merged
merged 4 commits into from
Nov 13, 2024

Conversation

pythagoraskitty
Copy link
Collaborator

@pythagoraskitty pythagoraskitty commented Nov 5, 2024

Spec update for #198.

This will make it possible for a worklet script to use shared storage data from an arbitrary origin other than the worklet script's origin or the invoking context's origin. The data origin would have to opt-in for data use to occur, as data processing could modify the data origin's data, send reports, and would affect its shared storage privacy budget.


Preview | Diff

Copy link
Collaborator

@xyaoinum xyaoinum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM % nits

@@ -178,6 +178,9 @@ spec: shared-storage-explainer; urlPrefix: https://github.com/WICG/shared-storag
spec: UUID; urlPrefix: https://www.ietf.org/rfc/rfc4122.txt
type: dfn
text: urn uuid; url: urn-uuid
spec: hr-time; urlPrefix: https://w3c.github.io/hr-time/
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just added this anchor because suddenly the spec stopped being able to find [=/current wall time=] on its own.

@pythagoraskitty
Copy link
Collaborator Author

@wanderview Could you please take a look? Thanks!

Copy link
Collaborator

@wanderview wanderview left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks pretty good, but I had a question.

@pythagoraskitty pythagoraskitty merged commit adc7252 into main Nov 13, 2024
1 check passed
@pythagoraskitty pythagoraskitty deleted the cammie-branch1 branch November 13, 2024 21:21
github-actions bot added a commit that referenced this pull request Nov 13, 2024
SHA: adc7252
Reason: push, by pythagoraskitty

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
aarongable pushed a commit to chromium/chromium that referenced this pull request Nov 27, 2024
We add support for custom data origins.

See WICG/shared-storage#198
and WICG/shared-storage#206.

Bug: 376278213
Change-Id: I7589ef08e68a8ec416d5ba1b986a81974f3e34d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5996831
Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
Commit-Queue: Cammie Smith Barnes <cammie@chromium.org>
Reviewed-by: Yao Xiao <yaoxia@chromium.org>
Reviewed-by: Ben Kelly <wanderview@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1388907}
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Nov 27, 2024
We add support for custom data origins.

See WICG/shared-storage#198
and WICG/shared-storage#206.

Bug: 376278213
Change-Id: I7589ef08e68a8ec416d5ba1b986a81974f3e34d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5996831
Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
Commit-Queue: Cammie Smith Barnes <cammie@chromium.org>
Reviewed-by: Yao Xiao <yaoxia@chromium.org>
Reviewed-by: Ben Kelly <wanderview@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1388907}
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Nov 27, 2024
We add support for custom data origins.

See WICG/shared-storage#198
and WICG/shared-storage#206.

Bug: 376278213
Change-Id: I7589ef08e68a8ec416d5ba1b986a81974f3e34d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5996831
Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
Commit-Queue: Cammie Smith Barnes <cammie@chromium.org>
Reviewed-by: Yao Xiao <yaoxia@chromium.org>
Reviewed-by: Ben Kelly <wanderview@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1388907}
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Nov 28, 2024
…in in createWorklet, a=testonly

Automatic update from web-platform-tests
Shared Storage: Support custom data origin in createWorklet

We add support for custom data origins.

See WICG/shared-storage#198
and WICG/shared-storage#206.

Bug: 376278213
Change-Id: I7589ef08e68a8ec416d5ba1b986a81974f3e34d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5996831
Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
Commit-Queue: Cammie Smith Barnes <cammie@chromium.org>
Reviewed-by: Yao Xiao <yaoxia@chromium.org>
Reviewed-by: Ben Kelly <wanderview@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1388907}

--

wpt-commits: d8ae479d673d32ab3dc69e08e1f0529c71769027
wpt-pr: 49405
i3roly pushed a commit to i3roly/firefox-dynasty that referenced this pull request Nov 30, 2024
…in in createWorklet, a=testonly

Automatic update from web-platform-tests
Shared Storage: Support custom data origin in createWorklet

We add support for custom data origins.

See WICG/shared-storage#198
and WICG/shared-storage#206.

Bug: 376278213
Change-Id: I7589ef08e68a8ec416d5ba1b986a81974f3e34d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5996831
Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
Commit-Queue: Cammie Smith Barnes <cammie@chromium.org>
Reviewed-by: Yao Xiao <yaoxia@chromium.org>
Reviewed-by: Ben Kelly <wanderview@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1388907}

--

wpt-commits: d8ae479d673d32ab3dc69e08e1f0529c71769027
wpt-pr: 49405
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Dec 1, 2024
…in in createWorklet, a=testonly

Automatic update from web-platform-tests
Shared Storage: Support custom data origin in createWorklet

We add support for custom data origins.

See WICG/shared-storage#198
and WICG/shared-storage#206.

Bug: 376278213
Change-Id: I7589ef08e68a8ec416d5ba1b986a81974f3e34d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5996831
Reviewed-by: Kenichi Ishibashi <bashichromium.org>
Commit-Queue: Cammie Smith Barnes <cammiechromium.org>
Reviewed-by: Yao Xiao <yaoxiachromium.org>
Reviewed-by: Ben Kelly <wanderviewchromium.org>
Reviewed-by: Mike West <mkwstchromium.org>
Cr-Commit-Position: refs/heads/main{#1388907}

--

wpt-commits: d8ae479d673d32ab3dc69e08e1f0529c71769027
wpt-pr: 49405

UltraBlame original commit: f74a9d288ec2e48dc358339b778fc966fb37d1e0
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Dec 1, 2024
…in in createWorklet, a=testonly

Automatic update from web-platform-tests
Shared Storage: Support custom data origin in createWorklet

We add support for custom data origins.

See WICG/shared-storage#198
and WICG/shared-storage#206.

Bug: 376278213
Change-Id: I7589ef08e68a8ec416d5ba1b986a81974f3e34d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5996831
Reviewed-by: Kenichi Ishibashi <bashichromium.org>
Commit-Queue: Cammie Smith Barnes <cammiechromium.org>
Reviewed-by: Yao Xiao <yaoxiachromium.org>
Reviewed-by: Ben Kelly <wanderviewchromium.org>
Reviewed-by: Mike West <mkwstchromium.org>
Cr-Commit-Position: refs/heads/main{#1388907}

--

wpt-commits: d8ae479d673d32ab3dc69e08e1f0529c71769027
wpt-pr: 49405

UltraBlame original commit: f74a9d288ec2e48dc358339b778fc966fb37d1e0
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Dec 1, 2024
…in in createWorklet, a=testonly

Automatic update from web-platform-tests
Shared Storage: Support custom data origin in createWorklet

We add support for custom data origins.

See WICG/shared-storage#198
and WICG/shared-storage#206.

Bug: 376278213
Change-Id: I7589ef08e68a8ec416d5ba1b986a81974f3e34d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5996831
Reviewed-by: Kenichi Ishibashi <bashichromium.org>
Commit-Queue: Cammie Smith Barnes <cammiechromium.org>
Reviewed-by: Yao Xiao <yaoxiachromium.org>
Reviewed-by: Ben Kelly <wanderviewchromium.org>
Reviewed-by: Mike West <mkwstchromium.org>
Cr-Commit-Position: refs/heads/main{#1388907}

--

wpt-commits: d8ae479d673d32ab3dc69e08e1f0529c71769027
wpt-pr: 49405

UltraBlame original commit: f74a9d288ec2e48dc358339b778fc966fb37d1e0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants