Changeset 184078 in webkit

Timestamp:

May 11, 2015, 4:14:33 AM (11 years ago)

Author:

Carlos Garcia Campos

Message:

Merge r182863 - Number.parseInt in nightly r182673 has wrong length
​https://bugs.webkit.org/show_bug.cgi?id=143657

Patch by Jordan Harband <​ljharb@gmail.com> on 2015-04-15
Reviewed by Benjamin Poulain.

Source/JavaScriptCore:

Correcting funciton length from 1, to 2, to match spec
​https://people.mozilla.org/~jorendorff/es6-draft.html#sec-number.parseint

• runtime/NumberConstructor.cpp:

(JSC::NumberConstructor::finishCreation):

LayoutTests:

• js/number-constructor-expected.txt:
• js/script-tests/number-constructor.js:

Location:

releases/WebKitGTK/webkit-2.8

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/js/number-constructor-expected.txt (modified) (1 diff)
• LayoutTests/js/script-tests/number-constructor.js (modified) (1 diff)
• Source/JavaScriptCore/ChangeLog (modified) (1 diff)
• Source/JavaScriptCore/runtime/NumberConstructor.cpp (modified) (1 diff)

releases/WebKitGTK/webkit-2.8/LayoutTests/ChangeLog

@@ +1 @@
+2015-04-15  Jordan Harband  <ljharb@gmail.com>
+
+        Number.parseInt in nightly r182673 has wrong length
+        https://bugs.webkit.org/show_bug.cgi?id=143657
+
+        Reviewed by Benjamin Poulain.
+
+        * js/number-constructor-expected.txt:
+        * js/script-tests/number-constructor.js:
+
 2015-04-14  Zalan Bujtas  <zalan@apple.com>
 

releases/WebKitGTK/webkit-2.8/LayoutTests/js/number-constructor-expected.txt

@@ -123 +123 @@
 PASS Number.parseFloat("200000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000") is Infinity
 PASS Number.parseFloat(foo) threw exception ReferenceError: Can't find variable: foo.
+PASS Number.parseInt.length is 2
 PASS Number.parseInt("0") is 0
 PASS Number.parseInt("-0") is -0

releases/WebKitGTK/webkit-2.8/LayoutTests/js/script-tests/number-constructor.js

@@ -131 +131 @@
 
 // parseInt
+shouldBe('Number.parseInt.length', '2');
 shouldBe('Number.parseInt("0")', '0');
 shouldBe('Number.parseInt("-0")', '-0');

releases/WebKitGTK/webkit-2.8/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2015-04-15  Jordan Harband  <ljharb@gmail.com>
+
+        Number.parseInt in nightly r182673 has wrong length
+        https://bugs.webkit.org/show_bug.cgi?id=143657
+
+        Reviewed by Benjamin Poulain.
+
+        Correcting funciton length from 1, to 2, to match spec
+        https://people.mozilla.org/~jorendorff/es6-draft.html#sec-number.parseint
+
+        * runtime/NumberConstructor.cpp:
+        (JSC::NumberConstructor::finishCreation):
+
 2015-04-14  Michael Saboff  <msaboff@apple.com>
 

releases/WebKitGTK/webkit-2.8/Source/JavaScriptCore/runtime/NumberConstructor.cpp

@@ -74 +74 @@
     putDirectNativeFunctionWithoutTransition(vm, numberPrototype->globalObject(), Identifier(&vm, "isSafeInteger"), 1, numberConstructorFuncIsSafeInteger, NoIntrinsic, DontEnum | Function);
     putDirectNativeFunctionWithoutTransition(vm, numberPrototype->globalObject(), Identifier(&vm, "parseFloat"), 1, globalFuncParseFloat, NoIntrinsic, DontEnum | Function);
-    putDirectNativeFunctionWithoutTransition(vm, numberPrototype->globalObject(), Identifier(&vm, "parseInt"), 1, globalFuncParseInt, NoIntrinsic, DontEnum | Function);
+    putDirectNativeFunctionWithoutTransition(vm, numberPrototype->globalObject(), Identifier(&vm, "parseInt"), 2, globalFuncParseInt, NoIntrinsic, DontEnum | Function);
 }