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

Unified Diff: chrome/browser/browsing_data/browsing_data_appcache_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_appcache_helper_unittest.cc
diff --git a/chrome/browser/browsing_data/browsing_data_appcache_helper_unittest.cc b/chrome/browser/browsing_data/browsing_data_appcache_helper_unittest.cc
index b06f9aea82fc57740fe10ad9833eb20be24c7070..4589574a10b627771059cc80d5ecbfc4ac61cc1c 100644
--- a/chrome/browser/browsing_data/browsing_data_appcache_helper_unittest.cc
+++ b/chrome/browser/browsing_data/browsing_data_appcache_helper_unittest.cc
@@ -31,6 +31,10 @@ class TestCompletionCallback {
} // namespace
class CannedBrowsingDataAppCacheHelperTest : public testing::Test {
+ public:
+ CannedBrowsingDataAppCacheHelperTest()
+ : thread_bundle_(content::TestBrowserThreadBundle::REAL_IO_THREAD) {}
+
content::TestBrowserThreadBundle thread_bundle_;
};
@@ -108,6 +112,28 @@ TEST_F(CannedBrowsingDataAppCacheHelperTest, Empty) {
ASSERT_TRUE(helper->empty());
}
+TEST_F(CannedBrowsingDataAppCacheHelperTest, Delete) {
+ TestingProfile profile;
+
+ GURL manifest1("http://example.com/manifest1.xml");
+ GURL manifest2("http://foo.example.com/manifest2.xml");
+ GURL manifest3("http://bar.example.com/manifest3.xml");
+
+ scoped_refptr<CannedBrowsingDataAppCacheHelper> helper(
+ new CannedBrowsingDataAppCacheHelper(&profile));
+
+ EXPECT_TRUE(helper->empty());
+ helper->AddAppCache(manifest1);
+ helper->AddAppCache(manifest2);
+ helper->AddAppCache(manifest3);
+ EXPECT_FALSE(helper->empty());
+ EXPECT_EQ(3u, helper->GetAppCacheCount());
+ helper->DeleteAppCacheGroup(manifest2);
+ EXPECT_EQ(2u, helper->GetAppCacheCount());
+ EXPECT_TRUE(helper->GetOriginAppCacheInfoMap().find(manifest2) ==
+ helper->GetOriginAppCacheInfoMap().end());
+}
+
TEST_F(CannedBrowsingDataAppCacheHelperTest, IgnoreExtensionsAndDevTools) {
TestingProfile profile;

Powered by Google App Engine
This is Rietveld 408576698