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

Assign "name" property to all anonymous functions #2299

Merged
merged 1 commit into from
Aug 15, 2019

Conversation

anba
Copy link
Contributor

@anba anba commented Aug 15, 2019

Implements the changes from tc39/ecma262#1490.

@leobalter
Copy link
Member

That's awesome, thank you!

cc @caiolima if you're working on the tests for class elements this ref PR in ECMA-262 might affect them as well.

@leobalter leobalter merged commit 59a1a01 into tc39:master Aug 15, 2019
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Aug 20, 2019
Update test262 to get the changes from <tc39/test262#2299>.

Differential Revision: https://phabricator.services.mozilla.com/D42158

--HG--
rename : js/src/tests/test262/language/statements/class/elements/prviate-accessor-is-visible-in-computed-properties.js => js/src/tests/test262/language/statements/class/elements/private-accessor-is-visible-in-computed-properties.js
rename : js/src/tests/test262/language/statements/class/elements/prviate-method-is-visible-in-computed-properties.js => js/src/tests/test262/language/statements/class/elements/private-method-is-visible-in-computed-properties.js
extra : moz-landing-system : lando
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Aug 20, 2019
Update test262 to get the changes from <tc39/test262#2299>.

Differential Revision: https://phabricator.services.mozilla.com/D42158

--HG--
rename : js/src/tests/test262/language/statements/class/elements/prviate-accessor-is-visible-in-computed-properties.js => js/src/tests/test262/language/statements/class/elements/private-accessor-is-visible-in-computed-properties.js
rename : js/src/tests/test262/language/statements/class/elements/prviate-method-is-visible-in-computed-properties.js => js/src/tests/test262/language/statements/class/elements/private-method-is-visible-in-computed-properties.js
extra : source : de51f104550cb6838cc24ae198a28828ff8b384a
xeonchen pushed a commit to xeonchen/gecko that referenced this pull request Aug 20, 2019
xeonchen pushed a commit to xeonchen/gecko that referenced this pull request Aug 20, 2019
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Oct 4, 2019
Update test262 to get the changes from <tc39/test262#2299>.

Differential Revision: https://phabricator.services.mozilla.com/D42158

UltraBlame original commit: de51f104550cb6838cc24ae198a28828ff8b384a
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Oct 4, 2019
Update test262 to get the changes from <tc39/test262#2299>.

Differential Revision: https://phabricator.services.mozilla.com/D42158

UltraBlame original commit: bc2a756c45543000ebae9cc907f8170326a950c5
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 4, 2019
Update test262 to get the changes from <tc39/test262#2299>.

Differential Revision: https://phabricator.services.mozilla.com/D42158

UltraBlame original commit: de51f104550cb6838cc24ae198a28828ff8b384a
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 4, 2019
Update test262 to get the changes from <tc39/test262#2299>.

Differential Revision: https://phabricator.services.mozilla.com/D42158

UltraBlame original commit: bc2a756c45543000ebae9cc907f8170326a950c5
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 4, 2019
Update test262 to get the changes from <tc39/test262#2299>.

Differential Revision: https://phabricator.services.mozilla.com/D42158

UltraBlame original commit: de51f104550cb6838cc24ae198a28828ff8b384a
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 4, 2019
Update test262 to get the changes from <tc39/test262#2299>.

Differential Revision: https://phabricator.services.mozilla.com/D42158

UltraBlame original commit: bc2a756c45543000ebae9cc907f8170326a950c5
@anba anba deleted the anon-name branch June 25, 2020 08:55
StenCalDabran added a commit to StenCalDabran/node that referenced this pull request Dec 15, 2020
Original commit message:

    Install "name" property on anonymous classes

    This is a normative PR that reached consensus at the June 2019 TC39:
    tc39/test262#2299

    Bug: v8:9646
    Change-Id: I8cb927b9e9231dfb71ebf47171205a096350e38b
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2360905
    Reviewed-by: Marja Hölttä <marja@chromium.org>
    Commit-Queue: Shu-yu Guo <syg@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#69460}

Refs:  v8/v8@048761a

Closes: nodejs#36038
targos pushed a commit to StenCalDabran/node that referenced this pull request Apr 24, 2021
Original commit message:

    Install "name" property on anonymous classes

    This is a normative PR that reached consensus at the June 2019 TC39:
    tc39/test262#2299

    Bug: v8:9646
    Change-Id: I8cb927b9e9231dfb71ebf47171205a096350e38b
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2360905
    Reviewed-by: Marja Hölttä <marja@chromium.org>
    Commit-Queue: Shu-yu Guo <syg@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#69460}

Refs: v8/v8@048761a

Closes: nodejs#36038
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants