-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Update Speculation Rules data with specs and Webview support #27935
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
Changes from 6 commits
3cce2cb
80d4e85
cd66ffb
7b5a42a
ff7083a
b53b584
3b85ccd
10e840f
a940d9a
411e415
de6e9aa
87acea4
bc9b695
2e50c7e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -1028,7 +1028,7 @@ | |||||
| }, | ||||||
| "status": { | ||||||
| "experimental": true, | ||||||
| "standard_track": false, | ||||||
| "standard_track": true, | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can we point to a spec where
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. So there's a PR open for that but not merged yet (hence why I didn't add the spec URL yet): So it was standard track in WICG, but is now moving to the actual, non-WICG standards (so also standard track) but is kind of in an in between state for now, but don't think that makes it non-standard. WDYT? Happy to update this with the spec url once it is merged to CSP spec btw.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Assuming that the PR will be merged soon, and knowing that you'll be following up, we can merge as is. In the future, when we derive the
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks. As i say it was in a WICG spec (which was included in browser-specs). But that spec has been closed off now as it’s being moved to the actual standards. My read of “standards track” is it’s on track to become a standard (which this is) even if not quite a standard yet. But I appreciate having a real spec is the only real evidence of that as a PR can be rejected. Bad timing with this PR. I should have updated this setting when it was in the old spec really as always was on the standards track. Anyway, thanks for the latitude!
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. (Gotcha, we removed these spec_urls in #27755.)
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ahhhh and that’s what flipped this to non standard. I wondered as didn’t think this was marked as such in the past. OK so all part of the fact we’re moving this!! |
||||||
| "deprecated": false | ||||||
| } | ||||||
| } | ||||||
|
|
||||||
Uh oh!
There was an error while loading. Please reload this page.