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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/browsing_data_remover_unittest.cc

Issue 7839029: QuotaManager::DeleteOriginData now allows deletion of specific QuotaClients (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: QuotaManager tests. Created 8 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
« no previous file with comments | « chrome/browser/browsing_data_remover.cc ('k') | webkit/quota/mock_quota_manager.h » ('j') | webkit/quota/mock_quota_manager.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browsing_data_remover_unittest.cc
diff --git a/chrome/browser/browsing_data_remover_unittest.cc b/chrome/browser/browsing_data_remover_unittest.cc
index 97a5c8e0a70a90b29e7a5a0bc89cb5cbd328834c..cacbc39f741df49aafbf2228ae37c796357c515f 100644
--- a/chrome/browser/browsing_data_remover_unittest.cc
+++ b/chrome/browser/browsing_data_remover_unittest.cc
@@ -42,6 +42,12 @@ const GURL kOrigin1(kTestkOrigin1);
const GURL kOrigin2(kTestkOrigin2);
const GURL kOrigin3(kTestkOrigin3);
+const quota::StorageType kTemporary = quota::kStorageTypeTemporary;
+const quota::StorageType kPersistent = quota::kStorageTypePersistent;
+
+const quota::QuotaClient::ID kClientFile = quota::QuotaClient::kFileSystem;
+const quota::QuotaClient::ID kClientDB = quota::QuotaClient::kIndexedDatabase;
+
class BrowsingDataRemoverTester : public BrowsingDataRemover::Observer {
public:
BrowsingDataRemoverTester()
@@ -202,31 +208,23 @@ class RemoveQuotaManagedDataTester : public BrowsingDataRemoverTester {
void PopulateTestQuotaManagedPersistentData(
quota::MockQuotaManager* manager) {
- manager->AddOrigin(kOrigin2, quota::kStorageTypePersistent,
- base::Time());
- manager->AddOrigin(kOrigin3, quota::kStorageTypePersistent,
+ manager->AddOrigin(kOrigin2, kPersistent, kClientFile, base::Time());
+ manager->AddOrigin(kOrigin3, kPersistent, kClientFile,
base::Time::Now() - base::TimeDelta::FromDays(1));
- EXPECT_FALSE(manager->OriginHasData(kOrigin1,
- quota::kStorageTypePersistent));
- EXPECT_TRUE(manager->OriginHasData(kOrigin2,
- quota::kStorageTypePersistent));
- EXPECT_TRUE(manager->OriginHasData(kOrigin3,
- quota::kStorageTypePersistent));
+ EXPECT_FALSE(manager->OriginHasData(kOrigin1, kPersistent, kClientFile));
+ EXPECT_TRUE(manager->OriginHasData(kOrigin2, kPersistent, kClientFile));
+ EXPECT_TRUE(manager->OriginHasData(kOrigin3, kPersistent, kClientFile));
}
void PopulateTestQuotaManagedTemporaryData(quota::MockQuotaManager* manager) {
- manager->AddOrigin(kOrigin1, quota::kStorageTypeTemporary,
- base::Time::Now());
- manager->AddOrigin(kOrigin3, quota::kStorageTypeTemporary,
+ manager->AddOrigin(kOrigin1, kTemporary, kClientFile, base::Time::Now());
+ manager->AddOrigin(kOrigin3, kTemporary, kClientFile,
base::Time::Now() - base::TimeDelta::FromDays(1));
- EXPECT_TRUE(manager->OriginHasData(kOrigin1,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(manager->OriginHasData(kOrigin2,
- quota::kStorageTypeTemporary));
- EXPECT_TRUE(manager->OriginHasData(kOrigin3,
- quota::kStorageTypeTemporary));
+ EXPECT_TRUE(manager->OriginHasData(kOrigin1, kTemporary, kClientFile));
+ EXPECT_FALSE(manager->OriginHasData(kOrigin2, kTemporary, kClientFile));
+ EXPECT_TRUE(manager->OriginHasData(kOrigin3, kTemporary, kClientFile));
}
private:
@@ -382,6 +380,30 @@ TEST_F(BrowsingDataRemoverTest, RemoveHistoryForLastHour) {
EXPECT_TRUE(tester->HistoryContainsURL(kOrigin2));
}
+TEST_F(BrowsingDataRemoverTest, QuotaClientMaskGeneration) {
+ EXPECT_EQ(quota::QuotaClient::kFileSystem,
+ BrowsingDataRemover::GenerateQuotaClientMask(
+ BrowsingDataRemover::REMOVE_FILE_SYSTEMS));
+ EXPECT_EQ(quota::QuotaClient::kDatabase,
+ BrowsingDataRemover::GenerateQuotaClientMask(
+ BrowsingDataRemover::REMOVE_WEBSQL));
+ EXPECT_EQ(quota::QuotaClient::kAppcache,
+ BrowsingDataRemover::GenerateQuotaClientMask(
+ BrowsingDataRemover::REMOVE_APPCACHE));
+ EXPECT_EQ(quota::QuotaClient::kIndexedDatabase,
+ BrowsingDataRemover::GenerateQuotaClientMask(
+ BrowsingDataRemover::REMOVE_INDEXEDDB));
+ EXPECT_EQ(quota::QuotaClient::kFileSystem |
+ quota::QuotaClient::kDatabase |
+ quota::QuotaClient::kAppcache |
+ quota::QuotaClient::kIndexedDatabase,
+ BrowsingDataRemover::GenerateQuotaClientMask(
+ BrowsingDataRemover::REMOVE_FILE_SYSTEMS |
+ BrowsingDataRemover::REMOVE_WEBSQL |
+ BrowsingDataRemover::REMOVE_APPCACHE |
+ BrowsingDataRemover::REMOVE_INDEXEDDB));
+}
+
TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForeverBoth) {
scoped_ptr<RemoveQuotaManagedDataTester> tester(
new RemoveQuotaManagedDataTester());
@@ -393,18 +415,18 @@ TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForeverBoth) {
EXPECT_EQ(BrowsingDataRemover::REMOVE_SITE_DATA &
~BrowsingDataRemover::REMOVE_PLUGIN_DATA, GetRemovalMask());
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypePersistent));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypePersistent));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypePersistent));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kPersistent,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kPersistent,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3, kPersistent,
+ kClientFile));
}
TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForeverOnlyTemporary) {
@@ -418,18 +440,18 @@ TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForeverOnlyTemporary) {
EXPECT_EQ(BrowsingDataRemover::REMOVE_SITE_DATA &
~BrowsingDataRemover::REMOVE_PLUGIN_DATA, GetRemovalMask());
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypePersistent));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypePersistent));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypePersistent));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kPersistent,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kPersistent,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3, kPersistent,
+ kClientFile));
}
TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForeverOnlyPersistent) {
@@ -443,18 +465,18 @@ TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForeverOnlyPersistent) {
EXPECT_EQ(BrowsingDataRemover::REMOVE_SITE_DATA &
~BrowsingDataRemover::REMOVE_PLUGIN_DATA, GetRemovalMask());
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypePersistent));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypePersistent));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypePersistent));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kPersistent,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kPersistent,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3, kPersistent,
+ kClientFile));
}
TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForeverNeither) {
@@ -468,18 +490,18 @@ TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForeverNeither) {
EXPECT_EQ(BrowsingDataRemover::REMOVE_SITE_DATA &
~BrowsingDataRemover::REMOVE_PLUGIN_DATA, GetRemovalMask());
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypePersistent));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypePersistent));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypePersistent));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kPersistent,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kPersistent,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3, kPersistent,
+ kClientFile));
}
TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForLastHour) {
@@ -493,18 +515,18 @@ TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForLastHour) {
EXPECT_EQ(BrowsingDataRemover::REMOVE_SITE_DATA &
~BrowsingDataRemover::REMOVE_PLUGIN_DATA, GetRemovalMask());
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypeTemporary));
- EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypePersistent));
- EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypePersistent));
- EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypePersistent));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kTemporary,
+ kClientFile));
+ EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin3, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kPersistent,
+ kClientFile));
+ EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin2, kPersistent,
+ kClientFile));
+ EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin3, kPersistent,
+ kClientFile));
}
TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForLastWeek) {
@@ -518,18 +540,18 @@ TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedDataForLastWeek) {
EXPECT_EQ(BrowsingDataRemover::REMOVE_SITE_DATA &
~BrowsingDataRemover::REMOVE_PLUGIN_DATA, GetRemovalMask());
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypePersistent));
- EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypePersistent));
- EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypePersistent));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kPersistent,
+ kClientFile));
+ EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin2, kPersistent,
+ kClientFile));
+ EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin3, kPersistent,
+ kClientFile));
}
TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedUnprotectedOrigins) {
@@ -549,18 +571,18 @@ TEST_F(BrowsingDataRemoverTest, RemoveQuotaManagedUnprotectedOrigins) {
EXPECT_EQ(BrowsingDataRemover::REMOVE_SITE_DATA &
~BrowsingDataRemover::REMOVE_PLUGIN_DATA, GetRemovalMask());
- EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypeTemporary));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1,
- quota::kStorageTypePersistent));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2,
- quota::kStorageTypePersistent));
- EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3,
- quota::kStorageTypePersistent));
+ EXPECT_TRUE(GetMockManager()->OriginHasData(kOrigin1, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3, kTemporary,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin1, kPersistent,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin2, kPersistent,
+ kClientFile));
+ EXPECT_FALSE(GetMockManager()->OriginHasData(kOrigin3, kPersistent,
+ kClientFile));
}
} // namespace
« no previous file with comments | « chrome/browser/browsing_data_remover.cc ('k') | webkit/quota/mock_quota_manager.h » ('j') | webkit/quota/mock_quota_manager.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698