base::Time should correctly handle -epoch time values from Javascript

base/time/time_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/time/time.h"
 
 #include <stdint.h>
 #include <time.h>
 #include <limits>
 #include <string>
@@ skipping 141 matching lines @@
 }
 
 // Test conversions to/from javascript time.
 TEST_F(TimeTest, JsTime) {
   Time epoch = Time::FromJsTime(0.0);
   EXPECT_EQ(epoch, Time::UnixEpoch());
   Time t = Time::FromJsTime(700000.3);
   EXPECT_EQ(700.0003, t.ToDoubleT());
   t = Time::FromDoubleT(800.73);
   EXPECT_EQ(800730.0, t.ToJsTime());
+
+  // Correctly convert |- epoch offset| which is valid time in javascript.
+  Time minusEpoch = Time::FromJsTime(-11644473600000.0);
+  EXPECT_EQ(-11644473600000, minusEpoch.ToJsTime());
+
+  // Check conversion of boundary javascript time values.
+  // See http://www.ecma-international.org/ecma-262/6.0/#sec-timeclip
+  EXPECT_EQ(-8.46e15, Time::FromJsTime(-8.46e15).ToJsTime());
+  EXPECT_EQ(8.46e15, Time::FromJsTime(8.46e15).ToJsTime());
 }
 
 #if defined(OS_POSIX)
 TEST_F(TimeTest, FromTimeVal) {
   Time now = Time::Now();
   Time also_now = Time::FromTimeVal(now.ToTimeVal());
   EXPECT_EQ(now, also_now);
 }
 #endif  // OS_POSIX
 
@@ skipping 1018 matching lines @@
 
 TEST(TimeTicksLogging, DoesNotMakeStreamBad) {
   std::ostringstream oss;
   oss << TimeTicks();
   EXPECT_TRUE(oss.good());
 }
 
 }  // namespace
 
 }  // namespace base