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

Update support data for Chrome Android (API's G through M) #18360

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 9 additions & 3 deletions api/HTMLAllCollection.json
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,9 @@
"chrome": {
"version_added": "4"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

The HTMLAllCollection tests are broken, fixing them in foolip/mdn-bcd-collector#2641.

Can you drop these changes?

},
"edge": {
"version_added": "12"
},
Expand Down Expand Up @@ -87,7 +89,9 @@
"chrome": {
"version_added": "6"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": {
"version_added": "12"
},
Expand Down Expand Up @@ -127,7 +131,9 @@
"chrome": {
"version_added": "4"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": {
"version_added": "12"
},
Expand Down
4 changes: 3 additions & 1 deletion api/HTMLMediaElement.json
Original file line number Diff line number Diff line change
Expand Up @@ -2792,7 +2792,9 @@
"chrome": {
"version_added": "49"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

Confirmed by https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/platform/runtime_enabled_features.json5;l=310-312;drc=8c770f58ec30541481bb61c02712c0fa2dbd685d

But there's a note from setSinkId that we should have here too:

Suggested change
"version_added": false
"version_added": false,
"notes": "Not available due to <a href='https://crbug.com/648286'>a limitation in Android</a>."

},
"edge": {
"version_added": "17"
},
Expand Down
12 changes: 9 additions & 3 deletions api/ImageBitmap.json
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,9 @@
"chrome": {
"version_added": "52"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

These tests are broken, fixed in foolip/mdn-bcd-collector#2642.

},
"edge": "mirror",
"firefox": {
"version_added": "46"
Expand Down Expand Up @@ -79,7 +81,9 @@
"chrome": {
"version_added": "50"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": "42"
Expand Down Expand Up @@ -113,7 +117,9 @@
"chrome": {
"version_added": "50"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": "42"
Expand Down
16 changes: 12 additions & 4 deletions api/InkPresenter.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,9 @@
"chrome": {
"version_added": "94"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

This interface has the wrong name in Chromium:
https://bugs.chromium.org/p/chromium/issues/detail?id=1404901

I've fixed the test in foolip/mdn-bcd-collector#2643 and expect that would lead to no change in the data here.

},
"edge": {
"version_added": "93"
},
Expand Down Expand Up @@ -41,7 +43,9 @@
"chrome": {
"version_added": "94"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": {
"version_added": "93"
},
Expand Down Expand Up @@ -76,7 +80,9 @@
"chrome": {
"version_added": "94"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": {
"version_added": "93"
},
Expand Down Expand Up @@ -111,7 +117,9 @@
"chrome": {
"version_added": "94"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": {
"version_added": "93"
},
Expand Down
4 changes: 3 additions & 1 deletion api/InstallEvent.json
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,9 @@
"chrome": {
"version_added": "40"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

There's a lot wrong with the data for InstallEvent and I think it should just be deleted.

In the spec the "install" event is an ExtendableEvent:
https://w3c.github.io/ServiceWorker/#service-worker-global-scope-install-event

InstallEvent is still in Chromium source but has no extra members, so one can still treat it as an ExtendableEvent.

},
"edge": {
"version_added": "17"
},
Expand Down
28 changes: 21 additions & 7 deletions api/InterventionReportBody.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@
"chrome": {
"version_added": "69"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

This is another case of #18359 (comment).

},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -41,7 +43,9 @@
"chrome": {
"version_added": "69"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -75,7 +79,9 @@
"chrome": {
"version_added": "69"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -109,7 +115,9 @@
"chrome": {
"version_added": "69"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -143,7 +151,9 @@
"chrome": {
"version_added": "69"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -177,7 +187,9 @@
"chrome": {
"version_added": "69"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -211,7 +223,9 @@
"chrome": {
"version_added": "69"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down
4 changes: 3 additions & 1 deletion api/KHR_parallel_shader_compile.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@
"chrome": {
"version_added": "76"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

Something was implemented in https://bugs.chromium.org/p/chromium/issues/detail?id=849576 and might work on Android given the right hardware. A lot of digging would be needed to confirm support or lack thereof.

},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down
8 changes: 6 additions & 2 deletions api/LaunchParams.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,9 @@
"chrome": {
"version_added": "102"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -39,7 +41,9 @@
"chrome": {
"version_added": "102"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down
8 changes: 6 additions & 2 deletions api/LaunchQueue.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,9 @@
"chrome": {
"version_added": "102"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -39,7 +41,9 @@
"chrome": {
"version_added": "102"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down
16 changes: 12 additions & 4 deletions api/Magnetometer.json
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,9 @@
"chrome": {
"version_added": "56"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

This is an interesting case!

Interpreting the current, there is no claim of support on Chrome Android, because of the experimental flag on the parent entry.

This runs into an old unresolved issue, #6509.

Did you use only the update-bcd script to produce this change? If so, can you file an issue against that?

},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -89,7 +91,9 @@
"chrome": {
"version_added": "56"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -129,7 +133,9 @@
"chrome": {
"version_added": "56"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -169,7 +175,9 @@
"chrome": {
"version_added": "56"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down
4 changes: 3 additions & 1 deletion api/MediaDevices.json
Original file line number Diff line number Diff line change
Expand Up @@ -409,7 +409,9 @@
"chrome": {
"version_added": "102"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down
4 changes: 3 additions & 1 deletion api/MediaRecorder.json
Original file line number Diff line number Diff line change
Expand Up @@ -786,7 +786,9 @@
"chrome": {
"version_added": "49"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": "25",
Expand Down
8 changes: 6 additions & 2 deletions api/MediaStreamTrack.json
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,9 @@
"chrome": {
"version_added": "102"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -294,7 +296,9 @@
"chrome": {
"version_added": "102"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down
12 changes: 9 additions & 3 deletions api/MediaStreamTrackProcessor.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@
"chrome": {
"version_added": "94"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

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

This turned out to be an interesting one.

https://mdn-bcd-collector.appspot.com/tests/api/MediaStreamTrackProcessor shows no support on desktop either, but note that the tests are for workers. The interface is exposed in only the window context in Chrome, but there's a ominous note in https://w3c.github.io/mediacapture-transform/#track-processor-interface:

There is WG consensus that the interface should be exposed on DedicatedWorker. There is no WG consensus on whether or not the interface should not be exposed on Window.

So what should this BCD represent? I think we should probably have a note that says it's only exposed in window, at the very least.

I've also commented at w3c/mediacapture-transform#66 (comment).

},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -42,7 +44,9 @@
"chrome": {
"version_added": "94"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down Expand Up @@ -76,7 +80,9 @@
"chrome": {
"version_added": "94"
},
"chrome_android": "mirror",
"chrome_android": {
"version_added": false
},
"edge": "mirror",
"firefox": {
"version_added": false
Expand Down
Loading