Updated JSON.stringify to newest version of ES5.

test/mjsunit/json.js

 // Copyright 2009 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 182 matching lines @@
 TestInvalid('"\\a invalid escape"');
 TestInvalid('"\\v invalid escape"');  // Valid JavaScript
 TestInvalid('"\\\' invalid escape"');  // Valid JavaScript
 TestInvalid('"\\x42 invalid escape"');  // Valid JavaScript
 TestInvalid('"\\u202 invalid escape"');
 TestInvalid('"\\012 invalid escape"');
 TestInvalid('"Unterminated string');
 TestInvalid('"Unterminated string\\"');
 TestInvalid('"Unterminated string\\\\\\"');
 
+// JavaScript RegExp literals not valid in JSON.
+TestInvalid('/true/');
+
 // Test bad JSON that would be good JavaScript (ES5).
-
 TestInvalid("{true:42}");
 TestInvalid("{false:42}");
 TestInvalid("{null:42}");
 TestInvalid("{'foo':42}");
 TestInvalid("{42:42}");
 TestInvalid("{0:42}");
 TestInvalid("{-1:42}");
 
 // Test for trailing garbage detection.
-
 TestInvalid('42 px');
 TestInvalid('42 .2');
 TestInvalid('42 2');
 TestInvalid('42 e1');
 TestInvalid('"42" ""');
 TestInvalid('"42" ""');
 TestInvalid('"" ""');
 TestInvalid('true ""');
 TestInvalid('false ""');
 TestInvalid('null ""');
@@ skipping 45 matching lines @@
 var singleton = [];
 var multiOccurrence = [singleton, singleton, singleton];
 assertEquals("[[],[],[]]", JSON.stringify(multiOccurrence));
 
 assertEquals('{"x":5,"y":6}', JSON.stringify({x:5,y:6}));
 assertEquals('{"x":5}', JSON.stringify({x:5,y:6}, ['x']));
 assertEquals('{\n "a": "b",\n "c": "d"\n}',
              JSON.stringify({a:"b",c:"d"}, null, 1));
 assertEquals('{"y":6,"x":5}', JSON.stringify({x:5,y:6}, ['y', 'x']));
 
+// The gap is capped at ten characters if specified as string.
+assertEquals('{\n          "a": "b",\n          "c": "d"\n}',
+              JSON.stringify({a:"b",c:"d"}, null, 
+                             "          /*characters after 10th*/"));
+
+//The gap is capped at ten characters if specified as number.
+assertEquals('{\n          "a": "b",\n          "c": "d"\n}',
+              JSON.stringify({a:"b",c:"d"}, null, 15));
+
+// Replaced wrapped primitives are unwrapped.
+function newx(k, v)  { return (k == "x") ? new v(42) : v; }
+assertEquals('{"x":"42"}', JSON.stringify({x: String}, newx));
+assertEquals('{"x":42}', JSON.stringify({x: Number}, newx));
+assertEquals('{"x":true}', JSON.stringify({x: Boolean}, newx));
+
 assertEquals(undefined, JSON.stringify(undefined));
 assertEquals(undefined, JSON.stringify(function () { }));
 

[Rico, 2010/02/03 12:57:23]

Maybe check for stringify on an array with undefined in it (to validate that we
do as 15.12.3-note 5)

[Lasse Reichstein, 2010/02/03 13:18:30]

Check added for behavior of undefined and functions in both arrays and objects.

+assertThrows("var a = [];a[0] = a;JSON.stringify(a);", TypeError);

[Rico, 2010/02/03 12:57:23]

Don't we check the same in line 269

[Lasse Reichstein, 2010/02/03 13:18:30]

We do. Removed.

+
 TestInvalid('1); throw "foo"; (1');
 
 var x = 0;
 eval("(1); x++; (1)");
 TestInvalid('1); x++; (1');