Changeset 39882 in webkit for trunk/JavaScriptCore/runtime/JSGlobalObjectFunctions.cpp

Timestamp:

Jan 13, 2009, 5:44:28 PM (16 years ago)

Author:

Beth Dakin

Message:

JavaScriptCore:

2009-01-13 Beth Dakin <Beth Dakin>

Reviewed by Darin Adler and Oliver Hunt.

<rdar://problem/6489314> REGRESSION: Business widget's front side
fails to render correctly when flipping widget

The problem here is that parseInt was parsing NaN as 0. This patch
corrects that by parsing NaN as NaN. This matches our old behavior
and Firefox.

• runtime/JSGlobalObjectFunctions.cpp: (JSC::globalFuncParseInt):

LayoutTests:

2009-01-13 Beth Dakin <Beth Dakin>

Reviewed by Darin Adler and Oliver Hunt.

Updated test and results for <rdar://problem/6489314> REGRESSION:
Business widget's front side fails to render correctly when
flipping widget

parseInt(NaN) should be NaN.

• fast/js/numeric-conversion-expected.txt:
• fast/js/resources/numeric-conversion.js:

File:

• trunk/JavaScriptCore/runtime/JSGlobalObjectFunctions.cpp (modified) (1 diff)

trunk/JavaScriptCore/runtime/JSGlobalObjectFunctions.cpp

@@ -303 +303 @@
             return value;
         double d = value->uncheckedGetNumber();
-        if (!isfinite(d))
-            return js0();
-        return jsNumber(exec, floor(d));
+        if (isfinite(d))
+            return jsNumber(exec, floor(d));
+        if (isnan(d))
+            return jsNaN(&exec->globalData());
+        return js0();
     }