Skip to content

Commit

Permalink
Sort out Safari 5.1 vs iOS 6 data
Browse files Browse the repository at this point in the history
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
  • Loading branch information
foolip committed Sep 21, 2022
1 parent 0614068 commit d398cc9
Show file tree
Hide file tree
Showing 8 changed files with 28 additions and 125 deletions.
44 changes: 11 additions & 33 deletions api/HTMLMarqueeElement.json
Original file line number Diff line number Diff line change
Expand Up @@ -68,9 +68,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -109,9 +107,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -186,9 +182,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -263,9 +257,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -304,9 +296,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -345,9 +335,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -386,9 +374,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -427,9 +413,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -578,9 +562,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -619,9 +601,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -660,9 +640,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down
8 changes: 2 additions & 6 deletions api/HashChangeEvent.json
Original file line number Diff line number Diff line change
Expand Up @@ -106,9 +106,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -148,9 +146,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down
4 changes: 1 addition & 3 deletions api/Window.json
Original file line number Diff line number Diff line change
Expand Up @@ -1338,9 +1338,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": {
"version_added": "≤37"
Expand Down
4 changes: 1 addition & 3 deletions css/properties/border-radius.json
Original file line number Diff line number Diff line change
Expand Up @@ -180,9 +180,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": {
"version_added": "≤37"
Expand Down
5 changes: 1 addition & 4 deletions css/properties/hyphens.json
Original file line number Diff line number Diff line change
Expand Up @@ -391,10 +391,7 @@
"version_added": "5.1",
"notes": "For English, Safari uses different en-GB and en-US dictionaries."
},
"safari_ios": {
"version_added": "6",
"notes": "For English, Safari uses different en-GB and en-US dictionaries."
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down
60 changes: 5 additions & 55 deletions css/types/image.json
Original file line number Diff line number Diff line change
Expand Up @@ -238,15 +238,7 @@
"version_added": "5.1"
}
],
"safari_ios": [
{
"version_added": "7"
},
{
"prefix": "-webkit-",
"version_added": "6"
}
],
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": [
{
Expand Down Expand Up @@ -420,19 +412,7 @@
]
}
],
"safari_ios": [
{
"version_added": "7"
},
{
"prefix": "-webkit-",
"version_added": "6",
"notes": [
"Safari 4 was supporting an experimental <code><a href='https://developer.apple.com/library/archive/documentation/InternetWeb/Conceptual/SafariVisualEffectsProgGuide/Gradients/Gradient.html'>-webkit-gradient(linear,…)</a></code> function. It is more limited than the later standard version: you cannot specify both a position and an angle like in <code>linear-gradient()</code>. This old outdated syntax is still supported for compatibility purposes.",
"Considers <code>&lt;angle&gt;</code> to start to the right, instead of the top. I.e. it considered an angle of <code>0deg</code> as a direction indicator pointing to the right."
]
}
],
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": [
{
Expand Down Expand Up @@ -676,16 +656,7 @@
"notes": "Safari 4 was supporting an experimental <code><a href='https://developer.apple.com/library/archive/documentation/InternetWeb/Conceptual/SafariVisualEffectsProgGuide/Gradients/Gradient.html'>-webkit-gradient(radial,…)</a></code> function. This old outdated syntax is still supported for compatibility purposes."
}
],
"safari_ios": [
{
"version_added": "7"
},
{
"prefix": "-webkit-",
"version_added": "6",
"notes": "Safari 4 was supporting an experimental <code><a href='https://developer.apple.com/library/archive/documentation/InternetWeb/Conceptual/SafariVisualEffectsProgGuide/Gradients/Gradient.html'>-webkit-gradient(radial,…)</a></code> function. This old outdated syntax is still supported for compatibility purposes."
}
],
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": [
{
Expand Down Expand Up @@ -966,19 +937,7 @@
]
}
],
"safari_ios": [
{
"version_added": "7"
},
{
"prefix": "-webkit-",
"version_added": "6",
"notes": [
"Safari 4 was supporting an experimental <code><a href='https://developer.apple.com/library/archive/documentation/InternetWeb/Conceptual/SafariVisualEffectsProgGuide/Gradients/Gradient.html'>-webkit-gradient(linear,…)</a></code> function. It is more limited than the later standard version: you cannot specify both a position and an angle like in <code>repeating-linear-gradient()</code>. This old outdated syntax is still supported for compatibility purposes.",
"Considers <code>&lt;angle&gt;</code> to start to the right, instead of the top. I.e. it considered an angle of <code>0deg</code> as a direction indicator pointing to the right."
]
}
],
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": [
{
Expand Down Expand Up @@ -1235,16 +1194,7 @@
"notes": "Safari 4 was supporting an experimental <code><a href='https://developer.apple.com/library/archive/documentation/InternetWeb/Conceptual/SafariVisualEffectsProgGuide/Gradients/Gradient.html'>-webkit-gradient(radial,…)</a></code> function. This old outdated syntax is still supported for compatibility purposes."
}
],
"safari_ios": [
{
"version_added": "7"
},
{
"prefix": "-webkit-",
"version_added": "6",
"notes": "Safari 4 was supporting an experimental <code><a href='https://developer.apple.com/library/archive/documentation/InternetWeb/Conceptual/SafariVisualEffectsProgGuide/Gradients/Gradient.html'>-webkit-gradient(radial,…)</a></code> function. This old outdated syntax is still supported for compatibility purposes."
}
],
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": [
{
Expand Down
24 changes: 6 additions & 18 deletions javascript/builtins/Object.json
Original file line number Diff line number Diff line change
Expand Up @@ -526,9 +526,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -990,9 +988,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -1038,9 +1034,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -1134,9 +1128,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -1328,9 +1320,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down Expand Up @@ -1514,9 +1504,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down
4 changes: 1 addition & 3 deletions javascript/statements.json
Original file line number Diff line number Diff line change
Expand Up @@ -287,9 +287,7 @@
"safari": {
"version_added": "5.1"
},
"safari_ios": {
"version_added": "6"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
"webview_android": "mirror"
},
Expand Down

0 comments on commit d398cc9

Please sign in to comment.