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

Add WebView versions for HTMLMediaElement API #8334

Merged
merged 2 commits into from
Dec 26, 2020

Conversation

queengooborg
Copy link
Contributor

This PR adds real values for WebView Android for the HTMLMediaElement API, based upon results from the mdn-bcd-collector project (v1.1.8). The collector obtains results based upon the latest WebView version (to determine if it is supported), then version numbers are copied from Chrome Android. Results are manually confirmed for accuracy.

Tests Used: https://mdn-bcd-collector.appspot.com/tests/api/HTMLMediaElement

This PR adds real values for WebView Android for the `HTMLMediaElement` API, based upon results from the [mdn-bcd-collector](https://mdn-bcd-collector.appspot.com) project (v1.1.8).  The collector obtains results based upon the latest WebView version (to determine if it is supported), then version numbers are copied from Chrome Android.  Results are manually confirmed for accuracy.

Tests Used: https://mdn-bcd-collector.appspot.com/tests/api/HTMLMediaElement
@github-actions github-actions bot added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Dec 25, 2020
@@ -357,7 +357,7 @@
"version_added": "4.0"
},
"webview_android": {
"version_added": "43"
"version_added": true
Copy link
Contributor

Choose a reason for hiding this comment

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

Why this change? A lot of the "43" in this file are wrong and would be fixed by testing the instance, maybe fix all of those first to ensure that no known-wrong data is being mirrored or overwritten in this PR? We should be able to fill in a real value here after the 43s and true values elsewhere are fixed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I actually don't know why I did this. I think I might've been thinking "I'll set it to true and fix these later", but we've got an issue open to track these down. Reverting these changes so that this PR is only increasing the amount of real values, not decreasing.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yep, let's get back to these in #7843.

@@ -1003,7 +1003,7 @@
"version_added": "5.0"
},
"webview_android": {
"version_added": "49"
"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.

Yep, confirmed by crbug.com/521319. I'll send a PR to link that issue after merging this.

@@ -3648,7 +3651,7 @@
"version_added": "5.0"
},
"webview_android": {
"version_added": "49"
"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 crbug.com/589500.

@foolip foolip merged commit 855c7e0 into mdn:master Dec 26, 2020
@queengooborg queengooborg deleted the api/HTMLMediaElement/webview branch December 26, 2020 08:39
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants