Closed
Bug 585208
Opened 14 years ago
Closed 14 years ago
reference to undefined Object.getOwnProperties
Categories
(Other Applications Graveyard :: Narcissus, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dherman, Assigned: dherman)
Details
Attachments
(1 file, 1 obsolete file)
6.32 KB,
patch
|
taustin
:
review+
|
Details | Diff | Splinter Review |
The proxy handler created in jsexec.newFunction makes reference to Object.getOwnProperties, which is not defined. The origin of that code: http://wiki.ecmascript.org/doku.php?id=harmony:proxies expects an implementation like this: http://code.google.com/p/es-lab/source/browse/trunk/src/traits/traits.js This needs to be defined somewhere in Narcissus if it's going to be used. Dave
Assignee | ||
Comment 1•14 years ago
|
||
Also, the |enumerateOwn| trap should be called keys. Dave
Assignee | ||
Comment 2•14 years ago
|
||
Assignee: nobody → dherman
Attachment #463720 -
Flags: review?(taustin)
Assignee | ||
Comment 3•14 years ago
|
||
Oh, Object.getPropertyDescriptor also doesn't exist. But it's pretty easy to implement. Updating the patch in a sec. Dave
Assignee | ||
Comment 4•14 years ago
|
||
Updated to implement getPropertyDescriptor as well. Dave
Attachment #463720 -
Attachment is obsolete: true
Attachment #463722 -
Flags: review?(taustin)
Attachment #463720 -
Flags: review?(taustin)
Updated•14 years ago
|
Attachment #463722 -
Flags: review?(taustin) → review+
Assignee | ||
Comment 5•14 years ago
|
||
http://hg.mozilla.org/tracemonkey/rev/8c22931336c5
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•