Revision: 12882
Author:   [email protected]
Date:     Wed Nov  7 04:12:52 2012
Log:      Remove check for recursion depth for JSON.stringify.

[email protected]
BUG=

Review URL: https://chromiumcodereview.appspot.com/11368119
http://code.google.com/p/v8/source/detail?r=12882

Modified:
 /branches/bleeding_edge/test/mjsunit/json-recursive.js

=======================================
--- /branches/bleeding_edge/test/mjsunit/json-recursive.js Thu Oct 25 05:39:41 2012 +++ /branches/bleeding_edge/test/mjsunit/json-recursive.js Wed Nov 7 04:12:52 2012
@@ -42,19 +42,14 @@
              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

Reply via email to