Closed
Bug 1096753
Opened 10 years ago
Closed 10 years ago
Array.isArray(new Proxy([], {})) should be false
Categories
(Core :: JavaScript: Standard Library, defect)
Core
JavaScript: Standard Library
Tracking
()
RESOLVED
DUPLICATE
of bug 1111785
People
(Reporter: Waldo, Unassigned)
References
(Blocks 1 open bug)
Details
ScriptedDirectProxyHandler needs an objectClassIs trap to fix this. (It actually needs one to handle revoked proxies correctly, independent of this issue.)
Comment 1•10 years ago
|
||
Jeff: A DXR link and and a general outline of the fix turns this in to a great first bug.
Flags: needinfo?(jwalden+bmo)
Comment 2•10 years ago
|
||
Relevant discussion thread on es-discuss: <https://esdiscuss.org/topic/array-isarray-new-proxy-should-be-false-bug-1096753>
Comment 3•10 years ago
|
||
This bug should await TC39 discussing the spec issue next week.
/be
Reporter | ||
Comment 4•10 years ago
|
||
es-discuss went ahead and made this into a more full-blown thing with property-gets, a canonical symbol, and a bit more. It's all rather more than seems to fit a mentored bug, at this point. (Tho it'd work fine for someone half a dozen patches into SpiderMonkey development, far enough to have a sense of SpiderMonkey idioms and such.)
Mentor: jwalden+bmo
Flags: needinfo?(jwalden+bmo)
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•