-
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
Update Safari versions for TypedArray #23319
Conversation
These 11 entries are changed by collector results: openwebdocs/mdn-bcd-results#1098 openwebdocs/mdn-bcd-results#1099 The collector shows no support in Safari 9.1 and support in 10.1, so source archeology was needed to decide between Safari 10 and 10.1. The 10 methods `copyWithin()`, `entries()`, `every()`, `fill()`, `filter()`, `find()`, `findIndex()`, `indexOf()`, `join()`, and `map()` were added in WebKit 602.1.6: WebKit/WebKit@881b0bf https://github.com/WebKit/WebKit/blob/881b0bfc93b6e6394676fbf712681792d72b2afa/Source/WebCore/Configurations/Version.xcconfig (See `JSTypedArrayViewPrototype::finishCreation`.) The static `of()` was added in WebKit 602.1.7: WebKit/WebKit@6fa0cb5 https://github.com/WebKit/WebKit/blob/6fa0cb524bcdd5091d1525846064f85782b9f0dd/Source/WebCore/Configurations/Version.xcconfig Both of those WebKit versions map to Safari 10. The Safari 9.1 data came from mdn#6480. That points to first commit above, but must have mapped it to a Safari version incorrectly. Note that some of the data was already set to Safari 10 going back to mdn#351.
d3a26e1
to
2e61e48
Compare
The correct version for typed arrays was carefully researched here: mdn/browser-compat-data#23319 This changes the Baseline low date by 1 year.
Safari 9.1 vs 10 was carefully researched here: mdn/browser-compat-data#23319 Where BCD is still wrong, we have to keep the data manually maintained until the next BCD release. After that it can be generated as .yml.dist. This changes the Baseline low date by 1 year.
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 for the archeology! Makes sense to me.
(We haven't invested much into the correctness of data prior to 2020, so errors like this can occur).
Yep, I understand. I only had to look into it because it was the keystone release, and it changes what the Baseline year is. |
Safari 9.1 vs 10 was carefully researched here: mdn/browser-compat-data#23319 Where BCD is still wrong, we have to keep the data manually maintained until the next BCD release. After that it can be generated as .yml.dist. This changes the Baseline low date by 1 year.
Here's a hint how much work it would be to go further into the past using the @openwebdocs collector project and the diff it spits out: openwebdocs/project#167 (comment). So, not impossible to work further into the past but definitely work. |
Thanks @Elchi3! My hunch is that it'll only be a sliver of this data that will end up affecting a Baseline low date between 2016 and 2020, but right now there's no way to know if a Baseline low date would be changed by a collector update. Just knowing when that's the case, then the proposed collector changes that could change a Baseline date could be reviewed. |
These 11 entries are changed by collector results:
openwebdocs/mdn-bcd-results#1098
openwebdocs/mdn-bcd-results#1099
The collector shows no support in Safari 9.1 and support in 10.1, so
source archeology was needed to decide between Safari 10 and 10.1.
The 10 methods
copyWithin()
,entries()
,every()
,fill()
,filter()
,find()
,findIndex()
,indexOf()
,join()
, andmap()
were added in WebKit 602.1.6:
WebKit/WebKit@881b0bf
https://github.com/WebKit/WebKit/blob/881b0bfc93b6e6394676fbf712681792d72b2afa/Source/WebCore/Configurations/Version.xcconfig
(See
JSTypedArrayViewPrototype::finishCreation
.)The static
of()
was added in WebKit 602.1.7:WebKit/WebKit@6fa0cb5
https://github.com/WebKit/WebKit/blob/6fa0cb524bcdd5091d1525846064f85782b9f0dd/Source/WebCore/Configurations/Version.xcconfig
Both of those WebKit versions map to Safari 10.
The Safari 9.1 data came from #6480.
That points to first commit above, but must have mapped it to a Safari
version incorrectly.
Note that some of the data was already set to Safari 10 going back to
#351.