Skip to content

Commit

Permalink
Fix incorrect webextension compat info (#22337)
Browse files Browse the repository at this point in the history
Fix incorrect BCD support info, and added impl_url to some entries that
have a bug tracking their implementation.

alarms: drop confusing compat info about past timers. The
version_added/version_removed ranges are hard to interpret. The affected
versions are ancient and the use of past timers is very unusual. Hence,
the simplest way to resolve the ambiguity is to drop the note. Ref:
https://bugzilla.mozilla.org/show_bug.cgi?id=1478694

clipboard API disabled in Firefox for Android:
https://searchfox.org/mozilla-central/rev/0e7394a77cdbe1df5e04a1d4171d6da67b57fa17/toolkit/components/extensions/parent/ext-clipboard.js#25-31

contextualIdentities not functional in Firefox for Android:
https://bugzilla.mozilla.org/show_bug.cgi?id=1638878

cookies: partitionKey supported in Chrome 119+: crbug.com/1225444
https://chromium.googlesource.com/chromium/src/+/45052d18b0f8dd0eb1a7492dc92acb8ba96aff64

dns: API restricted to Chrome dev
https://chromium.googlesource.com/chromium/src/+/603f4f94bc1d9bc363483bca56d31db492912d1a/extensions/common/api/_permission_features.json#231
"This API is only available in Chrome Dev. There are no foreseeable
 plans to move this API from the dev channel into Chrome stable."
https://developer.chrome.com/docs/extensions/reference/api/dns#availability

events: although it is more of an abstract interface, the methods are
supported in Firefox, e.g. as seen at:
https://searchfox.org/mozilla-central/rev/0e7394a77cdbe1df5e04a1d4171d6da67b57fa17/toolkit/components/extensions/schemas/events.json#45

tabs: tabs.detectLanguage supported in Firefox for Android 118+:
https://bugzilla.mozilla.org/show_bug.cgi?id=1817779

manifest/commands: _execute_browser_action introduced in:
https://bugzilla.mozilla.org/show_bug.cgi?id=1246034

manifest/commands: _execute_page_action introduced in:
https://bugzilla.mozilla.org/show_bug.cgi?id=1246035

manifest/background: Safari non-persistent background support mentioned in
https://developer.apple.com/documentation/safari-release-notes/safari-14_1-release-notes

manifest/content_scripts: match_origin_as_fallback added in:
https://chromium.googlesource.com/chromium/src/+/53396b76e9375
https://chromium.googlesource.com/chromium/src/+/53396b76e9375/chrome/VERSION

manifest/content_security_policy: isolated_world never shipped anywhere:
https://bugzilla.mozilla.org/show_bug.cgi?id=1594232#c5
  • Loading branch information
Rob--W authored Mar 3, 2024
1 parent 2250802 commit 461fd5b
Show file tree
Hide file tree
Showing 19 changed files with 238 additions and 81 deletions.
26 changes: 6 additions & 20 deletions webextensions/api/alarms.json
Original file line number Diff line number Diff line change
Expand Up @@ -90,26 +90,12 @@
"edge": {
"version_added": "79"
},
"firefox": [
{
"version_added": "45"
},
{
"version_added": "45",
"version_removed": "64",
"notes": "Alarms scheduled for a time in the past never fire."
}
],
"firefox_android": [
{
"version_added": "48"
},
{
"version_added": "48",
"version_removed": "64",
"notes": "Alarms scheduled for a time in the past never fire."
}
],
"firefox": {
"version_added": "45"
},
"firefox_android": {
"version_added": "48"
},
"opera": "mirror",
"safari": {
"version_added": "14"
Expand Down
5 changes: 4 additions & 1 deletion webextensions/api/clipboard.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,10 @@
"firefox": {
"version_added": "57"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"notes": "Method is defined but always rejects with an error."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand Down
8 changes: 6 additions & 2 deletions webextensions/api/commands.json
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,9 @@
"version_added": false,
"impl_url": "https://bugzil.la/1843866"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false
},
"opera": "mirror",
"safari": {
"version_added": false
Expand Down Expand Up @@ -177,7 +179,9 @@
"version_added": false,
"impl_url": "https://bugzil.la/1843866"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false
},
"opera": "mirror",
"safari": {
"version_added": false
Expand Down
90 changes: 75 additions & 15 deletions webextensions/api/contextualIdentities.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,11 @@
"firefox": {
"version_added": "53"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -32,7 +36,11 @@
"firefox": {
"version_added": "53"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -51,7 +59,11 @@
"firefox": {
"version_added": "57"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -70,7 +82,11 @@
"firefox": {
"version_added": "53"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -89,7 +105,11 @@
"firefox": {
"version_added": "57"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -108,7 +128,11 @@
"firefox": {
"version_added": "53"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -130,7 +154,11 @@
"version_added": "53",
"notes": "Before version 57, this method resolves its promise with <code>false</code> if the contextual identities feature is disabled."
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -154,7 +182,11 @@
"Before version 57, this method resolves its promise with <code>null</code> if the given identity was not found."
]
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -174,7 +206,11 @@
"firefox": {
"version_added": "57"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -194,7 +230,11 @@
"firefox": {
"version_added": "57"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -214,7 +254,11 @@
"firefox": {
"version_added": "57"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -234,7 +278,11 @@
"firefox": {
"version_added": "123"
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -255,7 +303,11 @@
"version_added": "53",
"notes": "Before version 57, this method resolves its promise with <code>false</code> if the contextual identities feature is disabled."
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -279,7 +331,11 @@
"Before version 57, this method resolves its promise with <code>null</code> if the given identity was not found."
]
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand All @@ -303,7 +359,11 @@
"Before version 57, this method resolves its promise with <code>null</code> if the given identity was not found."
]
},
"firefox_android": "mirror",
"firefox_android": {
"version_added": false,
"impl_url": "https://bugzil.la/1638878",
"notes": "<code>contextualIdentities</code> is defined but not functional in Firefox for Android."
},
"opera": "mirror",
"safari": {
"version_added": false
Expand Down
12 changes: 6 additions & 6 deletions webextensions/api/cookies.json
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@
"__compat": {
"support": {
"chrome": {
"version_added": false
"version_added": "119"
},
"edge": "mirror",
"firefox": {
Expand Down Expand Up @@ -248,7 +248,7 @@
"__compat": {
"support": {
"chrome": {
"version_added": false
"version_added": "119"
},
"edge": "mirror",
"firefox": {
Expand Down Expand Up @@ -322,7 +322,7 @@
"__compat": {
"support": {
"chrome": {
"version_added": false
"version_added": "119"
},
"edge": "mirror",
"firefox": {
Expand Down Expand Up @@ -390,7 +390,7 @@
"__compat": {
"support": {
"chrome": {
"version_added": false
"version_added": "119"
},
"edge": "mirror",
"firefox": {
Expand Down Expand Up @@ -454,7 +454,7 @@
"__compat": {
"support": {
"chrome": {
"version_added": false
"version_added": "119"
},
"edge": "mirror",
"firefox": {
Expand Down Expand Up @@ -629,7 +629,7 @@
"__compat": {
"support": {
"chrome": {
"version_added": false
"version_added": "119"
},
"edge": "mirror",
"firefox": {
Expand Down
22 changes: 21 additions & 1 deletion webextensions/api/dns.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,32 @@
"webextensions": {
"api": {
"dns": {
"__compat": {
"mdn_url": "https://developer.mozilla.org/docs/Mozilla/Add-ons/WebExtensions/API/dns",
"support": {
"chrome": {
"version_added": "preview",
"notes": "The required <code>dns</code> permission is only supported in Chrome Dev."
},
"edge": "mirror",
"firefox": {
"version_added": "60"
},
"firefox_android": "mirror",
"opera": "mirror",
"safari": {
"version_added": false
},
"safari_ios": "mirror"
}
},
"resolve": {
"__compat": {
"mdn_url": "https://developer.mozilla.org/docs/Mozilla/Add-ons/WebExtensions/API/dns/resolve",
"support": {
"chrome": {
"version_added": "18"
"version_added": "preview",
"notes": "The required <code>dns</code> permission is only supported in Chrome Dev."
},
"edge": "mirror",
"firefox": {
Expand Down
18 changes: 18 additions & 0 deletions webextensions/api/dom.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,24 @@
"webextensions": {
"api": {
"dom": {
"__compat": {
"mdn_url": "https://developer.mozilla.org/docs/Mozilla/Add-ons/WebExtensions/API/dom",
"support": {
"chrome": {
"version_added": "88"
},
"edge": "mirror",
"firefox": {
"version_added": false
},
"firefox_android": "mirror",
"opera": "mirror",
"safari": {
"version_added": false
},
"safari_ios": "mirror"
}
},
"openOrClosedShadowRoot": {
"__compat": {
"mdn_url": "https://developer.mozilla.org/docs/Mozilla/Add-ons/WebExtensions/API/dom/openOrClosedShadowRoot",
Expand Down
Loading

0 comments on commit 461fd5b

Please sign in to comment.