defineProperty on a index of a TypedArray should throw if configurable
Created attachment 283361 [details] Patch
Comment on attachment 283361 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=283361&action=review r=me with a suggestion > Source/JavaScriptCore/runtime/JSGenericTypedArrayViewInlines.h:351 > + if (!(descriptor.attributes() & DontDelete)) I think this is cleaner if you just call: "descriptor.configurable()" > Source/JavaScriptCore/runtime/JSGenericTypedArrayViewInlines.h:354 > + if (descriptor.attributes() & (DontEnum | ReadOnly)) ditto here but with descriptor.enumerable() and descriptor.writable()
Comment on attachment 283361 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=283361&action=review >> Source/JavaScriptCore/runtime/JSGenericTypedArrayViewInlines.h:351 >> + if (!(descriptor.attributes() & DontDelete)) > > I think this is cleaner if you just call: "descriptor.configurable()" Fixed. >> Source/JavaScriptCore/runtime/JSGenericTypedArrayViewInlines.h:354 >> + if (descriptor.attributes() & (DontEnum | ReadOnly)) > > ditto here but with descriptor.enumerable() and descriptor.writable() Fixed.
Committed r203096: <http://trac.webkit.org/changeset/203096>