Reviewers: Michael Starzinger,
Description:
Remove check for recursion depth for JSON.stringify.
[email protected]
BUG=
Please review this at https://chromiumcodereview.appspot.com/11368119/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M test/mjsunit/json-recursive.js
Index: test/mjsunit/json-recursive.js
diff --git a/test/mjsunit/json-recursive.js b/test/mjsunit/json-recursive.js
index
e9c15bbb2b9e18084bc1b42824026de46e25c61b..7a8c54774e3554fd69b74a7b2db7a7a40761dd1d
100644
--- a/test/mjsunit/json-recursive.js
+++ b/test/mjsunit/json-recursive.js
@@ -42,19 +42,14 @@ assertThrows(function() { rec(1, 2, 3, 4, 5, 6, 7, 8,
9, 0, 1, 2, 3, 4) },
RangeError);
-var depth1 = 1500;
-var depth2 = 10000;
+var depth = 10000;
var deepArray = [];
-for (var i = 0; i < depth1; i++) deepArray = [deepArray];
-JSON.stringify(deepArray);
-for (var i = depth1; i < depth2; i++) deepArray = [deepArray];
+for (var i = 0; i < depth; i++) deepArray = [deepArray];
assertThrows(function() { JSON.stringify(deepArray); }, RangeError);
var deepObject = {};
-for (var i = 0; i < depth1; i++) deepObject = { next: deepObject };
-JSON.stringify(deepObject);
-for (var i = depth1; i < depth2; i++) deepObject = { next: deepObject };
+for (var i = 0; i < depth; i++) deepObject = { next: deepObject };
assertThrows(function() { JSON.stringify(deepObject); }, RangeError);
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
