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

Replace most .__proto__ with Object.getPrototypeOf() #27394

Merged
merged 4 commits into from
Feb 2, 2021

Conversation

nayeemrmn
Copy link
Contributor

I left out the ones where .__proto__ appeared to be the subject of the test.

Ref: denoland/deno#4324. We hit this usage in Deno:

assert_equals(params.__proto__, URLSearchParams.prototype, 'expected URLSearchParams.prototype as prototype.');

Copy link
Member

@foolip foolip left a comment

Choose a reason for hiding this comment

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

Looks good, thanks! @nayeemrmn are you OK with the suggested rewording?

nayeemrmn and others added 2 commits January 30, 2021 00:19
…tor.html

Co-authored-by: Philip Jägenstedt <philip@foolip.org>
…tor.html

Co-authored-by: Philip Jägenstedt <philip@foolip.org>
@foolip
Copy link
Member

foolip commented Feb 2, 2021

Thanks @nayeemrmn! I didn't spot that you applied the suggestions 3 days ago, but I'll review and merge this now.

@foolip foolip merged commit 59d28c8 into web-platform-tests:master Feb 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants