Author: [EMAIL PROTECTED]
Date: Wed Oct 29 16:30:25 2008
New Revision: 645

Modified:
    branches/bleeding_edge/src/date-delay.js

Log:
Fix Chromium issue 3607.

Make Date::toLocaleDateString output be the same format as Safari.

Someone famililar with Date builtin object should check other functions.


Review URL: http://codereview.chromium.org/8875

Modified: branches/bleeding_edge/src/date-delay.js
==============================================================================
--- branches/bleeding_edge/src/date-delay.js    (original)
+++ branches/bleeding_edge/src/date-delay.js    Wed Oct 29 16:30:25 2008
@@ -517,6 +517,19 @@
  }


+var LongWeekDays =  
['Sunday', 'Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday'];
+var LongMonths =  
['January', 'February', 'March', 'April', 'May', 'June', 'July', 'August', 
'September', 'October', 'November', 'December'];
+
+
+function LongDateString(time) {
+  var YMD = FromJulianDay(Day(time) + kDayZeroInJulianDay);
+  return LongWeekDays[WeekDay(time)] + ', '
+      + LongMonths[YMD.month] + ' '
+      + TwoDigitString(YMD.date) + ', '
+      + YMD.year;
+}
+
+
  function TimeString(time) {
    return TwoDigitString(HourFromTime(time)) + ':'
        + TwoDigitString(MinFromTime(time)) + ':'
@@ -616,7 +629,9 @@

  // ECMA 262 - 15.9.5.6
  function DateToLocaleDateString() {
-  return DateToDateString.call(this);
+  var t = GetTimeFrom(this);
+  if ($isNaN(t)) return kInvalidDate;
+  return LongDateString(LocalTimeNoCheck(t));
  }



--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to