-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
Add StorageAccessHandle #33391
Add StorageAccessHandle #33391
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@bsmth have time for review? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've had a read through when you originally opened and checked again today, so I would leave a +1 to unblock you. Maybe @pepelsbey wants to have a look, but I think we can land it, IMO. Thank you!
Thanks! I could also use an eye on #33329 if you have time |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a suggestion on how to improve the readability of the parameters section. Otherwise, it looks good to me :)
Co-authored-by: Vadim Makeev <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@sideshowbarker have time to merge this one too? |
Description
This is the third of 3 PRs adding spec changes from https://privacycg.github.io/saa-non-cookie-storage/ launching in Chrome 125.
Motivation
StorageAccessHandle can be returned by requestStorageAccess when a types argument is provided to allow non-cookie unpartitioned storage access.
Additional details
https://chromestatus.com/feature/5175585823522816
Related issues and pull requests
Relates to mdn/mdn#543