Implement DefineOwnProperty for TypedArrays

test/mjsunit/element-accessor.js

Index: test/mjsunit/element-accessor.js
diff --git a/test/mjsunit/element-accessor.js b/test/mjsunit/element-accessor.js
index 5fba359c2a769cd5b46598bacc385c519ffa7762..e086a075d4b166cfc7e1cd270abe30c5573fd680 100644
--- a/test/mjsunit/element-accessor.js
+++ b/test/mjsunit/element-accessor.js
@@ -29,9 +29,14 @@
 })();
 
 (function () {
-  var o = new Int32Array();
-  Object.defineProperty(o, "0", {get: function(){}});
-  assertEquals(undefined, Object.getOwnPropertyDescriptor(o, "0"));

[caitp, 2016/11/17 15:24:42]

this old test was wrong per spec, does it still pass? This line should have
thrown instead of silently failing

[Henrique Ferreiro, 2016/11/18 09:27:07]

IntegerIndexedElementGet: Step 8. If index < 0 or index ≥ length, return
undefined.
GetOwnProperty: Step 3b ii. If value is undefined, return undefined

[caitp, 2016/11/18 14:47:51]

No, Object.defineProperty should throw here, because the index is out of range

+  var o = new Int32Array(1);
+  assertThrows('Object.defineProperty(o, "0", {get: function(){}})');

[Dan Ehrenberg, 2016/11/18 16:03:52]

Nit: For new tests, pass two arguments: the code to run (as a function, rather
than a string) and the class of the exception (here, TypeError).

+  assertEquals({
+    value: 0,
+    writable: true,
+    enumerable: true,
+    configurable: false
+  }, Object.getOwnPropertyDescriptor(o, "0"));
 })();
 
 (function() {