-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
updating Referrer Policy data #2858
Conversation
api/Request.json
Outdated
@@ -1273,47 +1273,47 @@ | |||
"mdn_url": "https://developer.mozilla.org/docs/Web/API/Request/referrerPolicy", | |||
"support": { | |||
"webview_android": { | |||
"version_added": null | |||
"version_added": "51" |
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.
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.
OK, updated.
api/SVGAElement.json
Outdated
@@ -204,43 +204,43 @@ | |||
"__compat": { | |||
"support": { | |||
"chrome": { | |||
"version_added": false | |||
"version_added": "38" |
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'm unable to confirm this. What's your information source?
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 was kinda guessing, by looking at this: https://www.chromestatus.com/features/5743723954569216 and assuming SVG attrs might be the same.
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 figured it out. You've got the Opera number here instead of the Chrome number.
api/WindowOrWorkerGlobalScope.json
Outdated
@@ -752,44 +752,44 @@ | |||
"description": "referrerPolicy", | |||
"support": { | |||
"chrome": { | |||
"version_added": "52" | |||
"version_added": "51" |
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'm also unable to confirm this.
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 think I'll change this one back.
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.
Fixed up the remaining version mismatch. Looks good to me now. Thanks for the PR! 👍
This PR updates and corrects the real values for Chromium (Chrome, Opera, Samsung Internet, WebView Android) for the `referrerPolicy` member of the `SVGAElement` API, based upon results from the [mdn-bcd-collector](https://mdn-bcd-collector.appspot.com) project (v6.1.0). Tests Used: https://mdn-bcd-collector.appspot.com/tests/api/SVGAElement/referrerPolicy _Check out the [collector's guide on how to review this PR](https://github.com/foolip/mdn-bcd-collector#reviewing-bcd-changes)._ This effectively reverts the SVGAElement changes of mdn#2858.
This effectively reverts the SVGAElement changes of #2858.
To find the data, I firstly looked at making it more consistent between pages, but I also looked up values/information at: