Closed
Bug 1063915
Opened 11 years ago
Closed 11 years ago
Make `dumpValue()` support symbols
Categories
(Core :: JavaScript Engine, enhancement)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla35
People
(Reporter: 446240525, Assigned: 446240525)
Details
Attachments
(1 file)
|
1.08 KB,
patch
|
evilpies
:
review+
|
Details | Diff | Splinter Review |
Actual Results:
js> dumpObject({[Symbol.for("foo")]: Symbol.iterator})
...
properties:
((js::Shape *) 0x105d5c970) enumerate unexpected value: slot 0 = unexpected value
Expected Results:
js> dumpObject({[Symbol.for("foo")]: Symbol.iterator})
...
properties:
((js::Shape *) 0x10545c970) enumerate Symbol.for("foo"): slot 0 = "Symbol.iterator"
Attachment #8485381 -
Flags: review?(jorendorff)
Updated•11 years ago
|
Attachment #8485381 -
Flags: review?(jorendorff) → review?(evilpies)
Comment 2•11 years ago
|
||
Comment on attachment 8485381 [details] [diff] [review]
bug1063915.patch
Stealing review. Looks good, thanks.
Attachment #8485381 -
Flags: review?(evilpies) → review+
Comment 3•11 years ago
|
||
Comment 4•11 years ago
|
||
Assignee: nobody → 446240525
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in
before you can comment on or make changes to this bug.
Description
•