Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1387)

Unified Diff: test/mjsunit/array-shift2.js

Issue 349073002: Remove SmartMove from array.js (Closed) Base URL: git://github.com/v8/v8.git@bleeding_edge
Patch Set: ws Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/mjsunit/array-shift2.js
diff --git a/test/mjsunit/array-shift2.js b/test/mjsunit/array-shift2.js
index 73d8cd4ff174602f7fd414b50139e1cdc25f02fd..aa7eb7476c1ab9c9855c948adbffddb2b056c797 100644
--- a/test/mjsunit/array-shift2.js
+++ b/test/mjsunit/array-shift2.js
@@ -12,7 +12,7 @@ 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 prope
Toon Verwaest 2014/06/23 08:42:36 Sounds good and correct. On 2014/06/22 23:59:19,
+assertEquals(["element 1","element 1"], test([{},,{}]));
%OptimizeFunctionOnNextCall(test);
-assertEquals(["element 1",0], test([{},,0]));
+assertEquals(["element 1","element 1"], test([{},,0]));

Powered by Google App Engine
This is Rietveld 408576698