Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix TypedArray.json for Safari #6480

Merged
merged 1 commit into from
Aug 5, 2020
Merged

Conversation

Constellation
Copy link
Contributor

@Constellation Constellation commented Aug 5, 2020

Other functions https://trac.webkit.org/changeset/190429/webkit
@@species https://trac.webkit.org/changeset/195460/webkit

A checklist to help your pull request get merged faster:

  • Summarize your changes
  • Data: link to resources that verify support information (such as browser's docs, changelogs, source control, bug trackers, and tests)
  • Data: if you tested something, describe how you tested with details like browser and version
  • Review the results of the linter and fix problems reported (If you need help, please ask in a comment!)
  • Link to related issues or pull requests, if any

@ghost ghost added the data:js 📟 Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript label Aug 5, 2020
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 👍

@Elchi3 Elchi3 merged commit 9f7a8b9 into mdn:master Aug 5, 2020
@Constellation Constellation deleted the typearray branch August 5, 2020 19:13
foolip added a commit to foolip/browser-compat-data that referenced this pull request Jun 10, 2024
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.
@foolip
Copy link
Collaborator

foolip commented Jun 10, 2024

I found my way here while trying to confirm that Safari 9.1 is the correct version for of() and ended up sending an update to the data in #23319.

Elchi3 pushed a commit that referenced this pull request Jun 10, 2024
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 #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.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
data:js 📟 Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants