Narrow cases where Sparse/Smart versions of Array methods are used

test/mjsunit/array-shift2.js

Index: test/mjsunit/array-shift2.js
diff --git a/test/mjsunit/array-shift2.js b/test/mjsunit/array-shift2.js
index 73d8cd4ff174602f7fd414b50139e1cdc25f02fd..f9fdf1d54eb78bc7b490a80f65501273c989e14f 100644
--- a/test/mjsunit/array-shift2.js
+++ b/test/mjsunit/array-shift2.js
@@ -12,7 +12,8 @@ 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]));

[Michael Starzinger, 2014/10/22 09:13:20]

suggestion: It would be nice to also verify that a[0] is an own property whereas
a[1] is not. Or alternatively we could verify that the setter in the prototype
has been invoked.

[adamk, 2014/10/22 18:49:44]

Done (added hasOwnProperty tests).

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