Remove SmartMove from array.js

test/mjsunit/array-shift2.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: --allow-natives-syntax
 
 Object.defineProperty(Array.prototype, "1", {
   get: function() { return "element 1"; },
   set: function(value) { }
 });
 function test(array) {
   array.shift();
   return array;
 }
-assertEquals(["element 1",2], test(["0",,2]));
-assertEquals(["element 1",{}], test([{},,{}]));
+assertEquals(["element 1","element 1"], test(["0",,2]));

[rafaelw, 2014/06/22 23:59:19]

This test was actually wrong. Given that the property on the proto is an
accessor, assignment should invoke setter, NOT write to the object.

It's not clear to me which of these three cases should remain. Maybe put the
"odd" object at element position 3.

[Toon Verwaest, 2014/06/23 08:42:36]

Sounds good and correct.

On 2014/06/22 23:59:19, rafaelw wrote:

+assertEquals(["element 1","element 1"], test([{},,{}]));
 %OptimizeFunctionOnNextCall(test);
-assertEquals(["element 1",0], test([{},,0]));
+assertEquals(["element 1","element 1"], test([{},,0]));