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 Chromium versions for javascript.builtins.Function.name #17762

Conversation

queengooborg
Copy link
Collaborator

This PR updates and corrects the real values for Chromium (Chrome, Opera, Samsung Internet, WebView Android) for the name member of the Function JavaScript builtin, based upon results from the mdn-bcd-collector project (v6.1.3).

Tests Used: https://mdn-bcd-collector.appspot.com/tests/javascript/builtins/Function/name

Check out the collector's guide on how to review this PR.

This PR updates and corrects the real values for Chromium (Chrome, Opera, Samsung Internet, WebView Android) for the `name` member of the `Function` JavaScript builtin, based upon results from the [mdn-bcd-collector](https://mdn-bcd-collector.appspot.com) project (v6.1.3).

Tests Used: https://mdn-bcd-collector.appspot.com/tests/javascript/builtins/Function/name

_Check out the [collector's guide on how to review this PR](https://github.com/foolip/mdn-bcd-collector#reviewing-bcd-changes)._
@queengooborg queengooborg added the data:js 📟 Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript label Sep 11, 2022
@foolip
Copy link
Collaborator

foolip commented Sep 13, 2022

Can you combine this with #17763 so that it's easy to see that the change was made at the same time in all WebKit-based browsers?

@queengooborg queengooborg deleted the javascript/builtins/Function/name/chrome-corrections branch December 19, 2022 06:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:js 📟 Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants