Implement DefineOwnProperty for TypedArrays

test/mjsunit/element-accessor.js

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Flags: --allow-natives-syntax
 
 (function () {
   var o = [];
   o.__proto__ = {};
 
@@ skipping 11 matching lines @@
   Object.defineProperty(o, "3", {
     get:function() { return 100; },
     set:function(v) { set = v; }});
 
   store(o, 3, 1000);
   assertEquals(1000, set);
   assertEquals(100, o[3]);
 })();
 
 (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() {
   function f() {
     var a = new Array();
     a[1] = 1.5;
     return a;
   }
 
   f();
   f();
   %OptimizeFunctionOnNextCall(f);
   var a = f();
   a[2] = 2;
   assertEquals(3, a.length);
 })();