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 Chromium versions for PluginArray API #7986

Merged
merged 2 commits into from
Dec 17, 2020

Conversation

queengooborg
Copy link
Collaborator

@queengooborg queengooborg commented Dec 17, 2020

This PR adds real values for Chromium (Chrome, Opera, Samsung Internet, WebView Android) for the PluginArray API, based upon results from the mdn-bcd-collector project (v1.1.6). Results are manually confirmed for accuracy.

Additionally, this adjusts a note on a few methods to match our typical wording, and copies it to all of the Chrome-based browsers.

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

This PR adds real values for Chromium (Chrome, Opera, Samsung Internet, WebView Android) for the `PluginArray` API, based upon results from the [mdn-bcd-collector](https://mdn-bcd-collector.appspot.com) project (v1.1.6).  Results are manually confirmed for accuracy.

Tests Used: https://mdn-bcd-collector.appspot.com/tests/api/PluginArray
@github-actions github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Dec 17, 2020
@@ -23,10 +23,10 @@
"version_added": null
},
"opera": {
"version_added": true
"version_added": "15"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder if this wasn't supported in Opera Presto. PluginArray is the type for navigator.plugins, can you update that entry at the same time and see if the values are the same? I suspect that the interface object just wasn't exposed, and that this needs to test an instance, which is simply navigator.plugins.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Looks like this is a case of unexposed interface, you're right. Adding a custom test, and manually updated the data.

@foolip foolip merged commit 6cd1272 into mdn:master Dec 17, 2020
@queengooborg queengooborg deleted the api/PluginArray-chrome branch December 17, 2020 15:11
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