802439 - js::SameValue isn't wrapper-aware

Status: RESOLVED INVALID

(Core :: JavaScript Engine, defect)

Description

[Jeff Walden [:Waldo]]

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'

Comment 1

[Jeff Walden [:Waldo]]

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...