Fix attributes of TypedArray properties

src/typedarray.js

Index: src/typedarray.js
diff --git a/src/typedarray.js b/src/typedarray.js
index ce52cdf15c0b50db12eb9c904fd1781ce30acf26..dec85421b3abb1c8882baea3ea4cda8b85970c0f 100644
--- a/src/typedarray.js
+++ b/src/typedarray.js
@@ -354,17 +354,18 @@ macro SETUP_TYPED_ARRAY(ARRAY_ID, NAME, ELEMENT_SIZE)
   %AddNamedProperty(GlobalNAME, "BYTES_PER_ELEMENT", ELEMENT_SIZE,
                     READ_ONLY | DONT_ENUM | DONT_DELETE);
   %AddNamedProperty(GlobalNAME.prototype,
-                    "constructor", global.NAME, DONT_ENUM);
+                    "constructor", global.NAME,
+                    READ_ONLY | DONT_ENUM | DONT_DELETE);

[adamk, 2015/07/16 19:16:12]

Why are you marking it this way? Am I missing another catch-all in the spec?

   %AddNamedProperty(GlobalNAME.prototype,
                     "BYTES_PER_ELEMENT", ELEMENT_SIZE,
                     READ_ONLY | DONT_ENUM | DONT_DELETE);
   utils.InstallGetter(GlobalNAME.prototype, "buffer", NAME_GetBuffer);
   utils.InstallGetter(GlobalNAME.prototype, "byteOffset", NAME_GetByteOffset,
-                      DONT_ENUM | DONT_DELETE);
+                      DONT_ENUM);

[adamk, 2015/07/16 19:16:12]

DONT_ENUM is the default, you can leave this off here and below.

   utils.InstallGetter(GlobalNAME.prototype, "byteLength", NAME_GetByteLength,
-                      DONT_ENUM | DONT_DELETE);
+                      DONT_ENUM);
   utils.InstallGetter(GlobalNAME.prototype, "length", NAME_GetLength,
-                      DONT_ENUM | DONT_DELETE);
+                      DONT_ENUM);
   utils.InstallGetter(GlobalNAME.prototype, symbolToStringTag,
                       TypedArrayGetToStringTag);
   utils.InstallFunctions(GlobalNAME.prototype, DONT_ENUM, [