js::SameValue isn't wrapper-aware

RESOLVED INVALID

Status

()

Core
JavaScript Engine
RESOLVED INVALID
5 years ago
5 years ago

People

(Reporter: Waldo, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

js> var obj = {};
js> var o = Object.defineProperty({}, "foo", { value: obj });
js> Object.defineProperty(o, "foo", { value: obj })
({})
js> Object.defineProperty(o, "foo", { value: wrap(obj) })
typein:4:0 TypeError: can't redefine non-configurable property 'foo'
Luke claims wrap() is unlike anything in the browser at all, which doesn't have same-compartment wrappers like wrap() creates, so this is probably invalid.  Which makes me wonder why we have wrap() at all, if it's just unspecified complexity that doesn't help move browser-based testing into shell-testable land...
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.