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

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

Issue 8888006: Make more JS files beter match the coding standard. Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Address review comments Created 9 years 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
« no previous file with comments | « test/mjsunit/arguments-read-and-assignment.js ('k') | test/mjsunit/array-constructor.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/array-concat.js
diff --git a/test/mjsunit/array-concat.js b/test/mjsunit/array-concat.js
index 97bd85aca2e646cd391558e0648458575d4e3c8b..904a626baf99b0eeb9cfb2467b60fe68910e44b8 100644
--- a/test/mjsunit/array-concat.js
+++ b/test/mjsunit/array-concat.js
@@ -183,7 +183,7 @@ assertEquals("Hello", c[0]);
assertEquals("Hello", c.toString());
// Check that concat preserves holes.
-var holey = [void 0,'a',,'c'].concat(['d',,'f',[0,,2],void 0])
+var holey = [void 0,'a',,'c'].concat(['d',,'f',[0,,2],void 0]);
assertEquals(9, holey.length); // hole in embedded array is ignored
for (var i = 0; i < holey.length; i++) {
if (i == 2 || i == 5) {
@@ -204,18 +204,20 @@ assertEquals([,2,1,3], r1);
// Make first array change length of second array.
Object.defineProperty(arr1, 0, {get: function() {
- arr2.push("X");
- return undefined;
- }, configurable: true})
+ arr2.push("X");
+ return undefined;
+ },
+ configurable: true});
var r2 = [].concat(arr1, arr2); // [undefined,2,1,3,"X"]
assertEquals([undefined,2,1,3,"X"], r2);
// Make first array change length of second array massively.
arr2.length = 2;
Object.defineProperty(arr1, 0, {get: function() {
- arr2[500000] = "X";
- return undefined;
- }, configurable: true})
+ arr2[500000] = "X";
+ return undefined;
+ },
+ configurable: true});
var r3 = [].concat(arr1, arr2); // [undefined,2,1,3,"X"]
var expected = [undefined,2,1,3];
expected[500000 + 2] = "X";
@@ -224,7 +226,7 @@ assertEquals(expected, r3);
var arr3 = [];
var trace = [];
-var expectedTrace = []
+var expectedTrace = [];
function mkGetter(i) { return function() { trace.push(i); }; }
arr3.length = 10000;
for (var i = 0; i < 100; i++) {
« no previous file with comments | « test/mjsunit/arguments-read-and-assignment.js ('k') | test/mjsunit/array-constructor.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698