Skip to content
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

Make Safari 5.1 mirror to iOS 5 (not 6) #17772

Merged
merged 2 commits into from
Sep 14, 2022
Merged

Conversation

foolip
Copy link
Collaborator

@foolip foolip commented Sep 13, 2022

Safari 5.1 has engine version 534.48, while iOS 5 has engine version 534.46. That's slightly before the Safari version, and this has caused mirroring so far to map Safari 5.1 to iOS 6. But 534.48 and 534.46 are close enough that we can consider them the same, and the Safari macOS release as an upstream for the iOS release.

Existing data mirroring from 5.1 is updated so that there are no changes to build/data.json.

This will allow a lot of 5.1+5 pairs to be mirrored in a follow-up PR.

@github-actions github-actions bot added data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:css 🎨 Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS data:js 📟 Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript scripts 📜 Issues or pull requests regarding the scripts in scripts/. labels Sep 13, 2022
@queengooborg queengooborg merged commit b610397 into mdn:main Sep 14, 2022
@foolip foolip deleted the mirror-safari-5.1 branch September 14, 2022 19:19
foolip added a commit to foolip/browser-compat-data that referenced this pull request Sep 15, 2022
@foolip
Copy link
Collaborator Author

foolip commented Sep 15, 2022

I've sent #17813 to burn down the changes made here, since the data was likely wrong.

queengooborg pushed a commit that referenced this pull request Sep 19, 2022
foolip added a commit to foolip/browser-compat-data that referenced this pull request Sep 20, 2022
This data ended up with Safari 2 / iOS 6 because of two PRs that don't
make sense when put together:
mdn#17772
mdn#17794

Mirror the data, matching CanvasPattern, createLinearGradient(), and
other related data.
foolip added a commit to foolip/browser-compat-data that referenced this pull request Sep 21, 2022
This revisits cases of Safari 5.1 + iOS 6 in the data preserved here:
mdn#17772

All features touched there are accounted for in the following...

CanvasGradient was fixed in these PRs:
mdn#17794
mdn#17843

api.HTMLMarqueeElement.*, WebKit version 534.14:
mdn#17011

api.HashChangeEvent.*, WebKit version 534.9:
WebKit/WebKit@a3f483b
https://github.com/WebKit/WebKit/blob/a3f483bbf1936c76f44387cde8fac903a86e69d0/WebCore/Configurations/Version.xcconfig

api.Window.error_event:
mdn#13053

css.properties.border-radius.percentages, WebKit version 534.7:
WebKit/WebKit@1c44c69
https://github.com/WebKit/WebKit/blob/1c44c69b77bddd43c8f5786fc27369cae9b9d3b2/WebCore/Configurations/Version.xcconfig

css.properties.hyphens.english was previously mirrored:
mdn#4636

css.selectors.left/right, WebKit 533.7?
mdn#4448

css.types.image.gradient.linear-gradient/radial-gradient, WebKit 534.16:
WebKit/WebKit@68c9486
https://github.com/WebKit/WebKit/blob/68c9486e0c50d5e468c9473685824b50d54ef15e/WebCore/Configurations/Version.xcconfig

css.types.image.gradient.repeating-linear-gradient/repeating-radial-gradient, WebKit 534.16:
WebKit/WebKit@b03353e
https://github.com/WebKit/WebKit/blob/b03353e8249b7a3a50cc42173bb4c8d6ea733ef3/WebCore/Configurations/Version.xcconfig

javascript.builtins.ArrayBuffer.slice:
mdn#17717 (comment)

javascript.builtins.Function.bind:
mdn#347

javascript.builtins.Object.freeze
javascript.builtins.Object.isExtensible
javascript.builtins.Object.isFrozen
javascript.builtins.Object.isSealed
javascript.builtins.Object.preventExtensions
javascript.builtins.Object.seal
javascript.statements.const
foolip added a commit to foolip/browser-compat-data that referenced this pull request Sep 21, 2022
This revisits cases of Safari 5.1 + iOS 6 in the data preserved here:
mdn#17772

All features touched there are accounted for in the following...

Handled in other PRs
====================

api.CanvasGradient:
mdn#17794
mdn#17843

css.selectors.left/right:
mdn#17852

Confirmed via source
====================

These features were confirmed to map to Safari 5.1 / iOS 5 via source,
landing in WebKit versions 534.7 through 534.16.

api.HTMLMarqueeElement, WebKit version 534.14:
mdn#17011

api.HashChangeEvent, WebKit version 534.9:
WebKit/WebKit@a3f483b
https://github.com/WebKit/WebKit/blob/a3f483bbf1936c76f44387cde8fac903a86e69d0/WebCore/Configurations/Version.xcconfig

css.properties.border-radius.percentages, WebKit version 534.7:
WebKit/WebKit@1c44c69
https://github.com/WebKit/WebKit/blob/1c44c69b77bddd43c8f5786fc27369cae9b9d3b2/WebCore/Configurations/Version.xcconfig

css.types.image.gradient.linear-gradient/radial-gradient, WebKit 534.16:
WebKit/WebKit@68c9486
https://github.com/WebKit/WebKit/blob/68c9486e0c50d5e468c9473685824b50d54ef15e/WebCore/Configurations/Version.xcconfig

css.types.image.gradient.repeating-linear-gradient/repeating-radial-gradient, WebKit 534.16:
WebKit/WebKit@b03353e
https://github.com/WebKit/WebKit/blob/b03353e8249b7a3a50cc42173bb4c8d6ea733ef3/WebCore/Configurations/Version.xcconfig

Confirmed with testing
======================

api.Window.error_event:
http://software.hixie.ch/utilities/js/live-dom-viewer/?saved=10758 was
run in iOS 4.2 and 5 to confirm iOS 5 is correct. Adapted from the test
in the PR that last updated this data:
mdn#13053

Previously mirrored
===================

These features were previously mirrored, so the choice of iOS 6 was not
deliberate.

css.properties.hyphens.english:
mdn#4636

javascript.builtins.Object.freeze,
javascript.builtins.Object.isExtensible,
javascript.builtins.Object.isFrozen,
javascript.builtins.Object.isSealed,
javascript.builtins.Object.preventExtensions,
javascript.builtins.Object.seal,
javascript.statements.const:
mdn#5483

Backported to Safari 5.1
========================

A bunch of JSC changes were backported to the Safari 5.1 release branch.
It's hard to determine if these were also backported to any iOS 5
release, so these are left as 5.1/6 pairs.

javascript.builtins.ArrayBuffer.slice:
mdn#17717 (comment)

javascript.builtins.Function.bind:
WebKit/WebKit@ec2129f
WebKit/WebKit@bb7691c
foolip added a commit to foolip/browser-compat-data that referenced this pull request Sep 21, 2022
This revisits cases of Safari 5.1 + iOS 6 in the data preserved here:
mdn#17772

All features touched there are accounted for in the following...

Handled in other PRs
====================

api.CanvasGradient:
mdn#17794
mdn#17843

css.selectors.left/right:
mdn#17852

Confirmed via source
====================

These features were confirmed to map to Safari 5.1 / iOS 5 via source,
landing in WebKit versions 534.7 through 534.16.

api.HTMLMarqueeElement, WebKit version 534.14:
mdn#17011

api.HashChangeEvent, WebKit version 534.9:
WebKit/WebKit@a3f483b
https://github.com/WebKit/WebKit/blob/a3f483bbf1936c76f44387cde8fac903a86e69d0/WebCore/Configurations/Version.xcconfig

css.properties.border-radius.percentages, WebKit version 534.7:
WebKit/WebKit@1c44c69
https://github.com/WebKit/WebKit/blob/1c44c69b77bddd43c8f5786fc27369cae9b9d3b2/WebCore/Configurations/Version.xcconfig

css.types.image.gradient.linear-gradient/radial-gradient, WebKit 534.16:
WebKit/WebKit@68c9486
https://github.com/WebKit/WebKit/blob/68c9486e0c50d5e468c9473685824b50d54ef15e/WebCore/Configurations/Version.xcconfig

css.types.image.gradient.repeating-linear-gradient/repeating-radial-gradient, WebKit 534.16:
WebKit/WebKit@b03353e
https://github.com/WebKit/WebKit/blob/b03353e8249b7a3a50cc42173bb4c8d6ea733ef3/WebCore/Configurations/Version.xcconfig

Confirmed with testing
======================

api.Window.error_event:
http://software.hixie.ch/utilities/js/live-dom-viewer/?saved=10758 was
run in iOS 4.2 and 5 to confirm iOS 5 is correct. Adapted from the test
in the PR that last updated this data:
mdn#13053

Previously mirrored
===================

These features were previously mirrored, so the choice of iOS 6 was not
deliberate.

css.properties.hyphens.english:
mdn#4636

javascript.builtins.Object.freeze,
javascript.builtins.Object.isExtensible,
javascript.builtins.Object.isFrozen,
javascript.builtins.Object.isSealed,
javascript.builtins.Object.preventExtensions,
javascript.builtins.Object.seal,
javascript.statements.const:
mdn#5483

Backported to Safari 5.1
========================

A bunch of JSC changes were backported to the Safari 5.1 release branch.
It's hard to determine if these were also backported to any iOS 5
release, so these are left as 5.1/6 pairs.

javascript.builtins.ArrayBuffer.slice:
mdn#17717 (comment)

javascript.builtins.Function.bind:
WebKit/WebKit@ec2129f
WebKit/WebKit@bb7691c
queengooborg pushed a commit that referenced this pull request Sep 21, 2022
This revisits cases of Safari 5.1 + iOS 6 in the data preserved here:
#17772

All features touched there are accounted for in the following...

Handled in other PRs
====================

api.CanvasGradient:
#17794
#17843

css.selectors.left/right:
#17852

Confirmed via source
====================

These features were confirmed to map to Safari 5.1 / iOS 5 via source,
landing in WebKit versions 534.7 through 534.16.

api.HTMLMarqueeElement, WebKit version 534.14:
#17011

api.HashChangeEvent, WebKit version 534.9:
WebKit/WebKit@a3f483b
https://github.com/WebKit/WebKit/blob/a3f483bbf1936c76f44387cde8fac903a86e69d0/WebCore/Configurations/Version.xcconfig

css.properties.border-radius.percentages, WebKit version 534.7:
WebKit/WebKit@1c44c69
https://github.com/WebKit/WebKit/blob/1c44c69b77bddd43c8f5786fc27369cae9b9d3b2/WebCore/Configurations/Version.xcconfig

css.types.image.gradient.linear-gradient/radial-gradient, WebKit 534.16:
WebKit/WebKit@68c9486
https://github.com/WebKit/WebKit/blob/68c9486e0c50d5e468c9473685824b50d54ef15e/WebCore/Configurations/Version.xcconfig

css.types.image.gradient.repeating-linear-gradient/repeating-radial-gradient, WebKit 534.16:
WebKit/WebKit@b03353e
https://github.com/WebKit/WebKit/blob/b03353e8249b7a3a50cc42173bb4c8d6ea733ef3/WebCore/Configurations/Version.xcconfig

Confirmed with testing
======================

api.Window.error_event:
http://software.hixie.ch/utilities/js/live-dom-viewer/?saved=10758 was
run in iOS 4.2 and 5 to confirm iOS 5 is correct. Adapted from the test
in the PR that last updated this data:
#13053

Previously mirrored
===================

These features were previously mirrored, so the choice of iOS 6 was not
deliberate.

css.properties.hyphens.english:
#4636

javascript.builtins.Object.freeze,
javascript.builtins.Object.isExtensible,
javascript.builtins.Object.isFrozen,
javascript.builtins.Object.isSealed,
javascript.builtins.Object.preventExtensions,
javascript.builtins.Object.seal,
javascript.statements.const:
#5483

Backported to Safari 5.1
========================

A bunch of JSC changes were backported to the Safari 5.1 release branch.
It's hard to determine if these were also backported to any iOS 5
release, so these are left as 5.1/6 pairs.

javascript.builtins.ArrayBuffer.slice:
#17717 (comment)

javascript.builtins.Function.bind:
WebKit/WebKit@ec2129f
WebKit/WebKit@bb7691c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:css 🎨 Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS data:js 📟 Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript scripts 📜 Issues or pull requests regarding the scripts in scripts/.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants