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

Side by Side Diff: net/cookies/cookie_store_unittest.h

Issue 1988343003: Access-time threshold unit failure in CookieMonster. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Alternate proposal. Created 4 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef NET_COOKIES_COOKIE_STORE_UNITTEST_H_ 5 #ifndef NET_COOKIES_COOKIE_STORE_UNITTEST_H_
6 #define NET_COOKIES_COOKIE_STORE_UNITTEST_H_ 6 #define NET_COOKIES_COOKIE_STORE_UNITTEST_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 1107 matching lines...) Expand 10 before | Expand all | Expand 10 after
1118 this->GetCookies(cs, this->https_www_google_.url())); 1118 this->GetCookies(cs, this->https_www_google_.url()));
1119 1119
1120 EXPECT_TRUE(this->SetCookie(cs, this->https_www_google_.url(), "A=B")); 1120 EXPECT_TRUE(this->SetCookie(cs, this->https_www_google_.url(), "A=B"));
1121 // The non-secure should overwrite the secure. 1121 // The non-secure should overwrite the secure.
1122 this->MatchCookieLines("A=B", 1122 this->MatchCookieLines("A=B",
1123 this->GetCookies(cs, this->http_www_google_.url())); 1123 this->GetCookies(cs, this->http_www_google_.url()));
1124 this->MatchCookieLines("D=E; A=B", 1124 this->MatchCookieLines("D=E; A=B",
1125 this->GetCookies(cs, this->https_www_google_.url())); 1125 this->GetCookies(cs, this->https_www_google_.url()));
1126 } 1126 }
1127 1127
1128 static const int kLastAccessThresholdMilliseconds = 200; 1128 static const base::TimeDelta kLastAccessThreshold =
1129 base::TimeDelta::FromMilliseconds(200);
Scott Hess - ex-Googler 2016/05/18 22:35:20 AFAICT this is only referenced by cookie_monster_u
Ryan Sleevi 2016/05/18 23:07:07 Agreed (if you want to move it there)
Scott Hess - ex-Googler 2016/05/18 23:14:29 Done.
1129 1130
1130 // Formerly NetUtilTest.CookieTest back when we used wininet's cookie handling. 1131 // Formerly NetUtilTest.CookieTest back when we used wininet's cookie handling.
1131 TYPED_TEST_P(CookieStoreTest, NetUtilCookieTest) { 1132 TYPED_TEST_P(CookieStoreTest, NetUtilCookieTest) {
1132 const GURL test_url("http://mojo.jojo.google.izzle/"); 1133 const GURL test_url("http://mojo.jojo.google.izzle/");
1133 1134
1134 CookieStore* cs = this->GetCookieStore(); 1135 CookieStore* cs = this->GetCookieStore();
1135 1136
1136 EXPECT_TRUE(this->SetCookie(cs, test_url, "foo=bar")); 1137 EXPECT_TRUE(this->SetCookie(cs, test_url, "foo=bar"));
1137 std::string value = this->GetCookies(cs, test_url); 1138 std::string value = this->GetCookies(cs, test_url);
1138 this->MatchCookieLines("foo=bar", value); 1139 this->MatchCookieLines("foo=bar", value);
(...skipping 269 matching lines...) Expand 10 before | Expand all | Expand 10 after
1408 OverwritePersistentCookie, 1409 OverwritePersistentCookie,
1409 CookieOrdering, 1410 CookieOrdering,
1410 GetAllCookiesAsync, 1411 GetAllCookiesAsync,
1411 DeleteCookieAsync, 1412 DeleteCookieAsync,
1412 DeleteCanonicalCookieAsync, 1413 DeleteCanonicalCookieAsync,
1413 DeleteSessionCookie); 1414 DeleteSessionCookie);
1414 1415
1415 } // namespace net 1416 } // namespace net
1416 1417
1417 #endif // NET_COOKIES_COOKIE_STORE_UNITTEST_H_ 1418 #endif // NET_COOKIES_COOKIE_STORE_UNITTEST_H_
OLDNEW
« net/cookies/cookie_monster.h ('K') | « net/cookies/cookie_monster_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698