Fixing Time::Max()'s behavior with Time::ToTimeT() and friends.

base/time_mac.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.h"
 
 #include <CoreFoundation/CFDate.h>
 #include <CoreFoundation/CFTimeZone.h>
 #include <mach/mach_time.h>
 #include <sys/time.h>
@@ skipping 36 matching lines @@
 
 // static
 Time Time::Now() {
   return FromCFAbsoluteTime(CFAbsoluteTimeGetCurrent());
 }
 
 // static
 Time Time::FromCFAbsoluteTime(CFAbsoluteTime t) {
   if (t == 0)
     return Time();  // Consider 0 as a null Time.
+  if (t == std::numeric_limits<CFAbsoluteTime>::max())
+    return Max();
   return Time(static_cast<int64>(
       (t + kCFAbsoluteTimeIntervalSince1970) * kMicrosecondsPerSecond) +
       kWindowsEpochDeltaMicroseconds);
 }
 
 CFAbsoluteTime Time::ToCFAbsoluteTime() const {
   if (is_null())
     return 0;  // Consider 0 as a null Time.
+  if (is_max())
+    return std::numeric_limits<CFAbsoluteTime>::max();
   return (static_cast<CFAbsoluteTime>(us_ - kWindowsEpochDeltaMicroseconds) /
       kMicrosecondsPerSecond) - kCFAbsoluteTimeIntervalSince1970;
 }
 
 // static
 Time Time::NowFromSystemTime() {
   // Just use Now() because Now() returns the system time.
   return Now();
 }
 
@@ skipping 85 matching lines @@
 TimeTicks TimeTicks::HighResNow() {
   return Now();
 }
 
 // static
 TimeTicks TimeTicks::NowFromSystemTraceTime() {
   return HighResNow();
 }
 
 }  // namespace base