-
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 Firefox support information for the Cookie Store API #36192
Conversation
Preview URLs External URLs (2)URL:
URL:
(comment last updated: 2024-10-09 12:24:26) |
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.
Looks good! Just a small suggestion ;)
Co-authored-by: Vadim Makeev <hi@pepelsbey.dev>
Co-authored-by: Vadim Makeev <hi@pepelsbey.dev>
This pull request has merge conflicts that must be resolved before it can be merged. |
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>
Description
A subset of the Cookie Store API is enabled in Firefox Nightly only, as of Firefox 132. This includes:
CookieStore
interface, but notpartitioned
in return values.CookieChangeEvent
interface, excludingpartitioned
properties.Window.cookieStore
ServiceWorkerGlobalScope.cookieStore
It doesn't include:
CookieStoreManager
interfaceExtendableCookieChangeEvent
interfaceServiceWorkerRegistration.cookies
ServiceWorkerGlobalScope
:cookiechange
event`I worked this out by looking at what WebIDL has been exposed in the implementation. See https://phabricator.services.mozilla.com/D215144.
This PR adds notes to the Firefox 132 rel notes and the Experimental features page to explain what support has bene shipped. I'm not sure if we need to think about adding information on how to handle cross-browser Cookie Store API code, or whether it is just case of some stuff doesn't work on Fx.
For the implementation bugs, see:
Motivation
Additional details
Related issues and pull requests
Documentation issue: #36122