Object.getOwnPropertyDescriptors should not create keys for undefined descriptors

RESOLVED FIXED in Firefox 50

Status

()

Core
JavaScript: Standard Library
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: evilpie, Assigned: evilpie)

Tracking

unspecified
mozilla50
Points:
---

Firefox Tracking Flags

(firefox50 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Not yet merged change to the spec: https://github.com/tc39/ecma262/pull/593
(Assignee)

Updated

2 years ago
Assignee: nobody → evilpies

Comment 1

2 years ago
This received consensus at the July 2016 TC39 meeting; please implement at your leisure.
(Assignee)

Comment 2

2 years ago
I am just going to wait until test262 is updated. After that this change should be super quick.
(Assignee)

Comment 3

2 years ago
Created attachment 8775091 [details] [diff] [review]
Implement change to O.getOwnPropertyDescriptors and update tests
Attachment #8775091 - Flags: review?(efaustbmo)

Comment 4

2 years ago
Comment on attachment 8775091 [details] [diff] [review]
Implement change to O.getOwnPropertyDescriptors and update tests

Review of attachment 8775091 [details] [diff] [review]:
-----------------------------------------------------------------

Yep, thanks.
Attachment #8775091 - Flags: review?(efaustbmo) → review+

Comment 5

2 years ago
Pushed by evilpies@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/71c43f3a49e4
Implement change to O.getOwnPropertyDescriptors and update tests. r=efaust

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/71c43f3a49e4
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.