Roll out change to HTMLParserIdioms.cpp from 82857
authorsimonjam@chromium.org <simonjam@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Apr 2012 22:53:20 +0000 (22:53 +0000)
committersimonjam@chromium.org <simonjam@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Apr 2012 22:53:20 +0000 (22:53 +0000)
https://bugs.webkit.org/show_bug.cgi?id=83402

Change 82857 causes a DCHECK on fast/forms/number/ValidityState-typeMismatch-number.html

Darin suggested we roll out this file here: https://bugs.webkit.org/show_bug.cgi?id=82857#c20

Unreviewed, rolling out change that broke tests.

* html/parser/HTMLParserIdioms.cpp:
(WebCore::parseToDoubleForNumberType):

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

Source/WebCore/ChangeLog
Source/WebCore/html/parser/HTMLParserIdioms.cpp

index f83fbd0..f7aa481 100644 (file)
@@ -1,3 +1,17 @@
+2012-04-06  James Simonsen  <simonjam@chromium.org>
+
+        Roll out change to HTMLParserIdioms.cpp from 82857
+        https://bugs.webkit.org/show_bug.cgi?id=83402
+
+        Change 82857 causes a DCHECK on fast/forms/number/ValidityState-typeMismatch-number.html
+
+        Darin suggested we roll out this file here: https://bugs.webkit.org/show_bug.cgi?id=82857#c20
+
+        Unreviewed, rolling out change that broke tests.
+
+        * html/parser/HTMLParserIdioms.cpp:
+        (WebCore::parseToDoubleForNumberType):
+
 2012-04-06  Ilya Sherman  <isherman@chromium.org>
 
         Allow site authors to override autofilled fields' colors.
index c2e53bd..b860202 100644 (file)
@@ -79,7 +79,10 @@ bool parseToDoubleForNumberType(const String& string, double* result)
     double value = string.toDouble(&valid);
     if (!valid)
         return false;
-    ASSERT(isfinite(value));
+
+    // NaN and infinity are considered valid by String::toDouble, but not valid here.
+    if (!isfinite(value))
+        return false;
 
     // Numbers are considered finite IEEE 754 single-precision floating point values.
     // See HTML5 2.5.4.3 `Real numbers.'