When parsing cookie expiration times, saturate out of range dates

net/cookies/cookie_util.h

Index: net/cookies/cookie_util.h
diff --git a/net/cookies/cookie_util.h b/net/cookies/cookie_util.h
index 0bf1452af864015331c1491ad50f78d1652e81dc..0822bd4b74f1be8b350587d9e1479054b561fb7a 100644
--- a/net/cookies/cookie_util.h
+++ b/net/cookies/cookie_util.h
@@ -37,8 +37,13 @@ NET_EXPORT bool GetCookieDomainWithString(const GURL& url,
 // i.e. it doesn't begin with a leading '.' character.
 NET_EXPORT bool DomainIsHostOnly(const std::string& domain_string);
 
-// Parses the string with the cookie time (very forgivingly).
-NET_EXPORT base::Time ParseCookieTime(const std::string& time_string);
+// Parses the string with the cookie expiration time (very forgivingly).
+// Returns the "null" time on failure.
+//
+// If the expiration date is outside the platform-specific range supported by
+// Time::FromUTCExplodeded(), then this will return the min/max non-null

[eroman, 2016/10/20 16:00:22]

Can you instead explicitly say Time(1) and Time::Max() ?

+// base::Time() value.
+NET_EXPORT base::Time ParseCookieExpirationTime(const std::string& time_string);
 
 // Convenience for converting a cookie origin (domain and https pair) to a URL.
 NET_EXPORT GURL CookieOriginToURL(const std::string& domain, bool is_https);