Make dates default to the local timezone if none specified

src/dateparser-inl.h

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_DATEPARSER_INL_H_
 #define V8_DATEPARSER_INL_H_
 
 #include "src/dateparser.h"
 
 namespace v8 {
 namespace internal {
 
 template <typename Char>
 bool DateParser::Parse(Vector<Char> str,
                        FixedArray* out,
                        UnicodeCache* unicode_cache) {
   DCHECK(out->length() >= OUTPUT_SIZE);
   InputReader<Char> in(unicode_cache, str);
   DateStringTokenizer<Char> scanner(&in);
   TimeZoneComposer tz;
   TimeComposer time;
   DayComposer day;
 
   // Specification:
   // Accept ES5 ISO 8601 date-time-strings or legacy dates compatible

[ulan, 2015/07/24 13:56:25]

Please update this comment to ES6.

[hichris123, 2015/07/24 15:10:04]

Done.

   // with Safari.
   // ES5 ISO 8601 dates:
   //   [('-'|'+')yy]yyyy[-MM[-DD]][THH:mm[:ss[.sss]][Z|(+|-)hh:mm]]
   //   where yyyy is in the range 0000..9999 and
   //         +/-yyyyyy is in the range -999999..+999999 -
   //           but -000000 is invalid (year zero must be positive),
   //         MM is in the range 01..12,
   //         DD is in the range 01..31,
   //         MM and DD defaults to 01 if missing,,
   //         HH is generally in the range 00..23, but can be 24 if mm, ss
   //           and sss are zero (or missing), representing midnight at the
   //           end of a day,
   //         mm and ss are in the range 00..59,
   //         sss is in the range 000..999,
   //         hh is in the range 00..23,
   //         mm, ss, and sss default to 00 if missing, and
-  //         timezone defaults to Z if missing
-  //           (following Safari, ISO actually demands local time).
+  //         timezone defaults to local time if missing
+  //           (following the ES6 standard vs. ES5).

[ulan, 2015/07/24 13:56:25]

We can now drop the part in parenthesis. 

[hichris123, 2015/07/24 15:10:04]

Done.

   //  Extensions:
   //   We also allow sss to have more or less than three digits (but at
   //   least one).
   //   We allow hh:mm to be specified as hhmm.
   // Legacy dates:
   //  Any unrecognized word before the first number is ignored.
   //  Parenthesized text is ignored.
   //  An unsigned number followed by ':' is a time value, and is
   //  added to the TimeComposer. A number followed by '::' adds a second
   //  zero as well. A number followed by '.' is also a time and must be
@@ skipping 238 matching lines @@
         if (!scanner->SkipSymbol(':')) return DateToken::Invalid();
         if (!scanner->Peek().IsFixedLengthNumber(2) ||
             !TimeComposer::IsMinute(scanner->Peek().number())) {
           return DateToken::Invalid();
         }
         tz->SetAbsoluteMinute(scanner->Next().number());
       }
     }
     if (!scanner->Peek().IsEndOfInput()) return DateToken::Invalid();
   }
-  // Successfully parsed ES5 Date Time String. Default to UTC if no TZ given.
-  if (tz->IsEmpty()) tz->Set(0);
+  // Successfully parsed ES5 Date Time String.
   day->set_iso_date();
   return DateToken::EndOfInput();
 }
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_DATEPARSER_INL_H_