Closed Bug 1054599 Opened 10 years ago Closed 8 years ago

Name property not set for ES6 method definitions + computed property names case

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 883377

People

(Reporter: gupta.rajagopal, Assigned: mrrrgn)

References

Details

Attachments

(1 file)

a="hey";
b={[a](){}};
b.hey.name === "hey" should evaluate to true
Attached patch namemethv0.diffSplinter Review
Very very raw patch - doesn't work.

What has to change:
During the clone, the name property should be set at the right place. The way I'm doing it, it's too late. The property is already frozen.

jorendorff and Waldo also wanted to eliminate the new bytecode to eliminate extra work in the jits.
Blocks: es6
Blocks: 883377
No longer blocks: es6
This has been sitting awhile and looks interesting to me: https://i.ytimg.com/vi/42CTe86UQjg/maxresdefault.jpg
Assignee: nobody → winter2718
I'll fix in bug 883377 at the same time.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: