Open Bug 1135997 Opened 10 years ago Updated 2 years ago

XrayWrapper<Base, Traits>::defineProperty: Use result.fail() to report certain errors.

Categories

(Core :: JavaScript Engine, defect)

x86_64
Linux
defect

Tracking

()

People

(Reporter: jorendorff, Unassigned)

References

(Blocks 1 open bug)

Details

XrayWrapper::defineProperty is stricter than ordinary Object.defineProperty about what sort of redefinition is permitted. But it doesn't report failure in any way when the rules are broken; it just silently doesn't define anything. There's even a comment: // We should technically report non-configurability in strict mode, but // doing that via JSAPI is a lot of trouble. With bug 1113369, it's not much trouble anymore. (In fact it's trivial, but it should be tested somehow.)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.