Fix Integer-overflow in base::Time::FromExploded.

base/time/time.h

Index: base/time/time.h
diff --git a/base/time/time.h b/base/time/time.h
index 12e43d9e7a02cf52711684f389a49ac5e51237f2..8a3610bde50ef2001b7a7dad9c7293baaba06b2e 100644
--- a/base/time/time.h
+++ b/base/time/time.h
@@ -545,7 +545,8 @@ class BASE_EXPORT Time : public time_internal::TimeBase<Time> {
 
   // Converts an exploded structure representing either the local time or UTC
   // into a Time class. Returns false on a failure when, for example, a day of
-  // month is set to 31 on a 28-30 day month.
+  // month is set to 31 on a 28-30 day month. Returns maximum possible

[miu, 2016/10/18 19:55:13]

IMO, if there is any overflow, this function should return false instead of just
returning "max possible time" because the conversion is not possible.

Or, can you point at specific code that requires this behavior?

[maksims (do not use this acc), 2016/10/19 16:41:04]

That's a typo. At first I thought it would be like this. It returns Time(0), of
course.

+  // platform dependent time on overflow.
   static bool FromUTCExploded(const Exploded& exploded,
                               Time* time) WARN_UNUSED_RESULT {
     return FromExploded(false, exploded, time);