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

Unified Diff: chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc

Issue 124183002: Implement delete methods for the various CannedBrowsingDataHelpers. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 11 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
Index: chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc
diff --git a/chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc b/chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc
index 7e02b8352514912eb78b27ffe250f0363abce969..faa861c0b242e37ece3eca64e46e05fd934e7b3f 100644
--- a/chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc
+++ b/chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc
@@ -217,6 +217,35 @@ TEST_F(BrowsingDataCookieHelperTest, DeleteCookie) {
base::RunLoop().RunUntilIdle();
}
+TEST_F(BrowsingDataCookieHelperTest, CannedDeleteCookie) {
+ CreateCookiesForTest();
+ scoped_refptr<CannedBrowsingDataCookieHelper> helper(
+ new CannedBrowsingDataCookieHelper(
+ testing_profile_->GetRequestContext()));
+
+ ASSERT_TRUE(helper->empty());
+
+ const GURL origin1("http://www.google.com");
+ const GURL origin2("http://www.gmail.google.com");
+ helper->AddChangedCookie(origin1, origin1, "A=1", net::CookieOptions());
+ helper->AddChangedCookie(origin2, origin2, "B=1", net::CookieOptions());
+
+ helper->StartFetching(
+ base::Bind(&BrowsingDataCookieHelperTest::FetchCallback,
+ base::Unretained(this)));
+ base::RunLoop().RunUntilIdle();
+
+ EXPECT_EQ(2u, helper->GetCookieCount());
+
+ helper->DeleteCookie(cookie_list_[0]);
+
+ EXPECT_EQ(1u, helper->GetCookieCount());
+ helper->StartFetching(
+ base::Bind(&BrowsingDataCookieHelperTest::DeleteCallback,
+ base::Unretained(this)));
+ base::RunLoop().RunUntilIdle();
+}
+
TEST_F(BrowsingDataCookieHelperTest, CannedDomainCookie) {
const GURL origin("http://www.google.com");
net::CookieList cookie;

Powered by Google App Engine
This is Rietveld 408576698