2010-12-28 Helder Correia <helder@sencha.com>
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Dec 2010 21:36:54 +0000 (21:36 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Dec 2010 21:36:54 +0000 (21:36 +0000)
        Reviewed by Sam Weinig.

        JSON.parse must exist as a function taking 2 parameters
        https://bugs.webkit.org/show_bug.cgi?id=51666

        Support for revivers was introduced in bug 26591, but the function
        length has since remained unchanged.

        Test: ietestcenter/Javascript/15.12.2-0-2.html

        * runtime/JSONObject.cpp:
2010-12-28  Helder Correia  <helder@sencha.com>

        Reviewed by Sam Weinig.

        JSON.parse must exist as a function taking 2 parameters
        https://bugs.webkit.org/show_bug.cgi?id=51666

        Support for revivers was introduced in bug 26591, but the function
        length has since remained unchanged.

        * ietestcenter/Javascript/15.12.2-0-2-expected.txt: Change expected
          result to success.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@74721 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/runtime/JSONObject.cpp
LayoutTests/ChangeLog
LayoutTests/ietestcenter/Javascript/15.12.2-0-2-expected.txt

index 94e522c..ef175c7 100644 (file)
@@ -1,3 +1,17 @@
+2010-12-28  Helder Correia  <helder@sencha.com>
+
+        Reviewed by Sam Weinig.
+
+        JSON.parse must exist as a function taking 2 parameters
+        https://bugs.webkit.org/show_bug.cgi?id=51666
+
+        Support for revivers was introduced in bug 26591, but the function
+        length has since remained unchanged.
+
+        Test: ietestcenter/Javascript/15.12.2-0-2.html
+
+        * runtime/JSONObject.cpp:
+
 2010-12-27  Jake Helfert  <jake@jakeonthenet.com>
 
         Reviewed and reworked by Darin Adler.
index b5477a1..ed9f1bd 100644 (file)
@@ -606,7 +606,7 @@ const ClassInfo JSONObject::info = { "JSON", 0, 0, ExecState::jsonTable };
 
 /* Source for JSONObject.lut.h
 @begin jsonTable
-  parse         JSONProtoFuncParse             DontEnum|Function 1
+  parse         JSONProtoFuncParse             DontEnum|Function 2
   stringify     JSONProtoFuncStringify         DontEnum|Function 1
 @end
 */
index f84ac8f..faecf54 100644 (file)
@@ -1,3 +1,16 @@
+2010-12-28  Helder Correia  <helder@sencha.com>
+
+        Reviewed by Sam Weinig.
+
+        JSON.parse must exist as a function taking 2 parameters
+        https://bugs.webkit.org/show_bug.cgi?id=51666
+
+        Support for revivers was introduced in bug 26591, but the function
+        length has since remained unchanged.
+
+        * ietestcenter/Javascript/15.12.2-0-2-expected.txt: Change expected
+          result to success.
+
 2010-12-23  Zhenyao Mo  <zmo@google.com>
 
         Reviewed by Kenneth Russell.
index d8dbf4c..4c384e6 100644 (file)
@@ -4,7 +4,7 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 
 PASS ES5Harness.preconditionPassed is true
-FAIL ES5Harness.testPassed should be true (of type boolean). Was undefined (of type undefined).
+PASS ES5Harness.testPassed is true
 PASS successfullyParsed is true
 
 TEST COMPLETE