Summary: | Object.preventExtensions(window) should throw a TypeError | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||
Component: | DOM | Assignee: | Chris Dumez <cdumez> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ashvayka, cdumez, commit-queue, ggaren, mark.lam, rniwa, saam | ||||
Priority: | P2 | Keywords: | WebExposed | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
URL: | https://tc39.github.io/ecma262/#sec-object.preventextensions | ||||||
See Also: |
https://bugs.webkit.org/show_bug.cgi?id=161486 https://bugs.webkit.org/show_bug.cgi?id=219270 |
||||||
Attachments: |
|
Description
Chris Dumez
2016-09-02 16:16:47 PDT
Created attachment 287845 [details]
Patch
Comment on attachment 287845 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=287845&action=review The code here doesn’t make sense to me. > Source/WebCore/bindings/js/JSDOMWindowCustom.cpp:349 > + throwTypeError(exec, scope, ASCIILiteral("Cannot prevent extensions on this object")); Should the message be more specific than "this object"? Comment on attachment 287845 [details] Patch Clearing flags on attachment: 287845 Committed r205404: <http://trac.webkit.org/changeset/205404> All reviewed patches have been landed. Closing bug. |