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..94acc5c6c3533e2349c0be4f22632d079fb45391 100644
--- a/test/mjsunit/element-accessor.js
+++ b/test/mjsunit/element-accessor.js
@@ -29,9 +29,15 @@
 })();
 
 (function () {
-  var o = new Int32Array();
-  Object.defineProperty(o, "0", {get: function(){}});
-  assertEquals(undefined, Object.getOwnPropertyDescriptor(o, "0"));
+  var o = new Int32Array(1);
+  assertThrows(
+      () => Object.defineProperty(o, '0', {get: function() {}}), TypeError);

[Dan Ehrenberg, 2016/11/28 23:01:40]

Looks like this case started to throw when it previously didn't.

[caitp, 2016/11/28 23:10:10]

Defining accessors on typed arrays used to fail silently, but this was not spec
behaviour. all other impls ive tried match the spec here.

[Dan Ehrenberg, 2016/11/28 23:13:13]

OK, if we're not the first to ship this fix, I'm fine without the UseCounter.

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