Index: chrome/browser/browsing_data/browsing_data_remover_browsertest.cc |
diff --git a/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc b/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc |
index cdde107acf5150318ad758bb1e6027ee61d0798e..1fc217c3222f8ad4468b6bc67374aa1b6f96b4cd 100644 |
--- a/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc |
+++ b/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc |
@@ -15,6 +15,7 @@ |
#include "chrome/browser/browsing_data/browsing_data_remover_factory.h" |
#include "chrome/browser/browsing_data/browsing_data_remover_test_util.h" |
#include "chrome/browser/browsing_data/cache_counter.h" |
+#include "chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.h" |
#include "chrome/browser/chrome_notification_types.h" |
#include "chrome/browser/external_protocol/external_protocol_handler.h" |
#include "chrome/browser/profiles/profile.h" |
@@ -118,9 +119,9 @@ class BrowsingDataRemoverBrowserTest : public InProcessBrowserTest { |
BrowsingDataRemover* remover = |
BrowsingDataRemoverFactory::GetForBrowserContext(browser()->profile()); |
BrowsingDataRemoverCompletionObserver completion_observer(remover); |
- remover->RemoveAndReply( |
- base::Time(), base::Time::Max(), remove_mask, |
- BrowsingDataHelper::UNPROTECTED_WEB, &completion_observer); |
+ remover->RemoveAndReply(base::Time(), base::Time::Max(), remove_mask, |
+ BrowsingDataRemover::ORIGIN_TYPE_UNPROTECTED_WEB, |
+ &completion_observer); |
completion_observer.BlockUntilCompletion(); |
} |
@@ -132,8 +133,8 @@ class BrowsingDataRemoverBrowserTest : public InProcessBrowserTest { |
BrowsingDataRemoverCompletionObserver completion_observer(remover); |
remover->RemoveWithFilterAndReply( |
base::Time(), base::Time::Max(), remove_mask, |
- BrowsingDataHelper::UNPROTECTED_WEB, std::move(filter_builder), |
- &completion_observer); |
+ BrowsingDataRemover::ORIGIN_TYPE_UNPROTECTED_WEB, |
+ std::move(filter_builder), &completion_observer); |
completion_observer.BlockUntilCompletion(); |
} |
@@ -194,7 +195,7 @@ class BrowsingDataRemoverTransportSecurityStateBrowserTest |
// Test BrowsingDataRemover for downloads. |
IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverBrowserTest, Download) { |
DownloadAnItem(); |
- RemoveAndWait(BrowsingDataRemover::REMOVE_DOWNLOADS); |
+ RemoveAndWait(BrowsingDataRemover::DATA_TYPE_DOWNLOADS); |
VerifyDownloadCount(0u); |
} |
@@ -208,7 +209,7 @@ IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverBrowserTest, DownloadProhibited) { |
prefs->SetBoolean(prefs::kAllowDeletingBrowserHistory, false); |
DownloadAnItem(); |
- RemoveAndWait(BrowsingDataRemover::REMOVE_DOWNLOADS); |
+ RemoveAndWait(BrowsingDataRemover::DATA_TYPE_DOWNLOADS); |
VerifyDownloadCount(1u); |
} |
#endif |
@@ -223,7 +224,7 @@ IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverBrowserTest, Database) { |
RunScriptAndCheckResult("insertRecord('text')", "done"); |
RunScriptAndCheckResult("getRecords()", "text"); |
- RemoveAndWait(BrowsingDataRemover::REMOVE_SITE_DATA); |
+ RemoveAndWait(ChromeBrowsingDataRemoverDelegate::DATA_TYPE_SITE_DATA); |
ui_test_utils::NavigateToURL(browser(), url); |
RunScriptAndCheckResult("createTable()", "done"); |
@@ -254,7 +255,7 @@ IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverBrowserTest, Cache) { |
std::unique_ptr<BrowsingDataFilterBuilder> filter_builder = |
BrowsingDataFilterBuilder::Create(BrowsingDataFilterBuilder::WHITELIST); |
filter_builder->AddOrigin(url::Origin(url1)); |
- RemoveWithFilterAndWait(BrowsingDataRemover::REMOVE_CACHE, |
+ RemoveWithFilterAndWait(BrowsingDataRemover::DATA_TYPE_CACHE, |
std::move(filter_builder)); |
// After the partial deletion, the cache should be smaller but still nonempty. |
@@ -265,12 +266,12 @@ IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverBrowserTest, Cache) { |
filter_builder = |
BrowsingDataFilterBuilder::Create(BrowsingDataFilterBuilder::WHITELIST); |
filter_builder->AddOrigin(url::Origin(url1)); |
- RemoveWithFilterAndWait(BrowsingDataRemover::REMOVE_CACHE, |
+ RemoveWithFilterAndWait(BrowsingDataRemover::DATA_TYPE_CACHE, |
std::move(filter_builder)); |
EXPECT_EQ(new_size, GetCacheSize()); |
// Delete the remaining data. |
- RemoveAndWait(BrowsingDataRemover::REMOVE_CACHE); |
+ RemoveAndWait(BrowsingDataRemover::DATA_TYPE_CACHE); |
// The cache is empty. |
EXPECT_EQ(0, GetCacheSize()); |
@@ -285,7 +286,7 @@ IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverBrowserTest, |
ExternalProtocolHandler::BlockState block_state = |
ExternalProtocolHandler::GetBlockState("tel", profile); |
ASSERT_EQ(ExternalProtocolHandler::BLOCK, block_state); |
- RemoveAndWait(BrowsingDataRemover::REMOVE_SITE_DATA); |
+ RemoveAndWait(ChromeBrowsingDataRemoverDelegate::DATA_TYPE_SITE_DATA); |
block_state = ExternalProtocolHandler::GetBlockState("tel", profile); |
ASSERT_EQ(ExternalProtocolHandler::UNKNOWN, block_state); |
} |
@@ -293,7 +294,7 @@ IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverBrowserTest, |
// Verify that TransportSecurityState data is cleared for REMOVE_CACHE. |
IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverTransportSecurityStateBrowserTest, |
ClearTransportSecurityState) { |
- RemoveAndWait(BrowsingDataRemover::REMOVE_CACHE); |
+ RemoveAndWait(BrowsingDataRemover::DATA_TYPE_CACHE); |
base::RunLoop run_loop; |
BrowserThread::PostTaskAndReply( |
BrowserThread::IO, FROM_HERE, |
@@ -309,7 +310,7 @@ IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverTransportSecurityStateBrowserTest, |
// set. |
IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverTransportSecurityStateBrowserTest, |
PreserveTransportSecurityState) { |
- RemoveAndWait(BrowsingDataRemover::REMOVE_SITE_DATA); |
+ RemoveAndWait(ChromeBrowsingDataRemoverDelegate::DATA_TYPE_SITE_DATA); |
base::RunLoop run_loop; |
BrowserThread::PostTaskAndReply( |
BrowserThread::IO, FROM_HERE, |