Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(380)

Unified Diff: net/cookies/cookie_monster_store_test.cc

Issue 2424443002: When parsing cookie expiration times, saturate out of range dates (Closed)
Patch Set: Response to comments Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/cookies/canonical_cookie.cc ('k') | net/cookies/cookie_util.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/cookies/cookie_monster_store_test.cc
diff --git a/net/cookies/cookie_monster_store_test.cc b/net/cookies/cookie_monster_store_test.cc
index 27e6dd6232df37f6854f5d44006258ffdfed7b93..1375134a16c0e581c1007e8ce96b22788809da00 100644
--- a/net/cookies/cookie_monster_store_test.cc
+++ b/net/cookies/cookie_monster_store_test.cc
@@ -125,9 +125,9 @@ std::unique_ptr<CanonicalCookie> BuildCanonicalCookie(
// functions. Would be nice to export them, and re-use here.
EXPECT_FALSE(pc.HasMaxAge());
EXPECT_TRUE(pc.HasPath());
- base::Time cookie_expires = pc.HasExpires()
- ? cookie_util::ParseCookieTime(pc.Expires())
- : base::Time();
+ base::Time cookie_expires =
+ pc.HasExpires() ? cookie_util::ParseCookieExpirationTime(pc.Expires())
+ : base::Time();
std::string cookie_path = pc.Path();
return CanonicalCookie::Create(url, pc.Name(), pc.Value(), url.host(),
« no previous file with comments | « net/cookies/canonical_cookie.cc ('k') | net/cookies/cookie_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698