In Array.of and Array.from, fall back to DefineOwnProperty

test/mjsunit/harmony/array-from.js

 // Copyright 2014 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: --harmony-arrays
 (function() {
 
 assertEquals(1, Array.from.length);
 
 function assertArrayLikeEquals(value, expected, type) {
@@ skipping 130 matching lines @@
 
 testArrayFrom(Array, Array);
 testArrayFrom(null, Array);
 testArrayFrom({}, Array);
 testArrayFrom(Object, Object);
 testArrayFrom(Other, Other);
 testArrayFrom(Math.cos, Array);
 testArrayFrom(Math.cos.bind(Math), Array);
 testArrayFrom(boundFn, boundFn);
 
+// Assert that [[DefineOwnProperty]] is used in ArrayFrom, meaning a
+// setter isn't called, and a failed [[DefineOwnProperty]] will throw.
+var setterCalled = 0;
+function exotic() {
+  Object.defineProperty(this,  '0', {
+    get: function() { return 2; },
+    set: function() { setterCalled++; }
+  });
+}
+// Check that exotic was defined right
+var instance = new exotic();
+assertEquals(2, instance[0]);
+instance[0] = 1;
+assertEquals(2, instance[0]);
+assertEquals(1, setterCalled);
+// Accessor properties can't be overwritten with DefineOwnProperty
+assertThrows(function () { Array.from.call(exotic, [1]); }, TypeError);
+assertEquals(1, setterCalled);
+
 })();

[arv (Not doing code reviews), 2015/06/11 13:04:01]

I see that there is already a test for accessors on Array.prototype.

[Dan Ehrenberg, 2015/06/11 16:27:25]

Not sure what you're referring to, but I added a test.

[arv (Not doing code reviews), 2015/06/11 16:41:30]

Ignore me.