Changeset 14206 in webkit for trunk/JavaScriptCore

Timestamp:

May 5, 2006, 8:09:59 AM (19 years ago)

Author:

darin

Message:

JavaScriptCore:

• ​http://bugzilla.opendarwin.org/show_bug.cgi?id=8722 IE compatibility fix in date parsing

• kjs/date_object.cpp: (KJS::parseDate): Merged change that George Staikos provided from KDE 3.4.3 branch that allows day values of 0 and values that are > 1000.

LayoutTests:

• test update for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=8722 IE compatibility fix in date parsing

• fast/js/kde/resources/Date.js: Added test cases that George Staikos provided that test dates with day values of 0, 55, and 452.

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• kjs/date_object.cpp (modified) (3 diffs)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2006-05-05  Darin Adler  <[email protected]>
+
+        - http://bugzilla.opendarwin.org/show_bug.cgi?id=8722
+          IE compatibility fix in date parsing
+
+        * kjs/date_object.cpp: (KJS::parseDate): Merged change that George Staikos provided
+        from KDE 3.4.3 branch that allows day values of 0 and values that are > 1000.
+
 2006-05-04  Anders Carlsson  <[email protected]>
 

trunk/JavaScriptCore/kjs/date_object.cpp

@@ -1020 +1020 @@
         return NaN;
 
-    if (day < 1)
+    if (day < 0)
         return NaN;
 
@@ -1026 +1026 @@
     if (day > 31) {
         // ### where is the boundary and what happens below?
-        if (!(*dateString == '/' && day >= 1000))
+        if (*dateString != '/')
             return NaN;
         // looks like a YYYY/MM/DD date
@@ -1042 +1042 @@
             return NaN;
         dateString = newPosStr;
-    } else if (*dateString == '/' && day <= 12 && month == -1) {
+    } else if (*dateString == '/' && month == -1) {
         dateString++;
         // This looks like a MM/DD/YYYY date, not an RFC date.