| Index: chrome/browser/chrome_content_browser_client_unittest.cc
|
| diff --git a/chrome/browser/chrome_content_browser_client_unittest.cc b/chrome/browser/chrome_content_browser_client_unittest.cc
|
| index e09b076f2ee53467a45de8d768b8289720b45971..9fff4cb7497c82a45ca5f10d029644d798ffd4e0 100644
|
| --- a/chrome/browser/chrome_content_browser_client_unittest.cc
|
| +++ b/chrome/browser/chrome_content_browser_client_unittest.cc
|
| @@ -19,7 +19,6 @@
|
| #include "base/strings/utf_string_conversions.h"
|
| #include "build/build_config.h"
|
| #include "chrome/browser/browsing_data/browsing_data_helper.h"
|
| -#include "chrome/browser/browsing_data/browsing_data_remover_factory.h"
|
| #include "chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.h"
|
| #include "chrome/browser/browsing_data/mock_browsing_data_remover_delegate.h"
|
| #include "chrome/browser/search_engines/template_url_service_factory.h"
|
| @@ -30,6 +29,7 @@
|
| #include "components/variations/variations_associated_data.h"
|
| #include "components/version_info/version_info.h"
|
| #include "content/public/browser/browsing_data_filter_builder.h"
|
| +#include "content/public/browser/browsing_data_remover.h"
|
| #include "content/public/browser/navigation_controller.h"
|
| #include "content/public/browser/navigation_entry.h"
|
| #include "content/public/browser/storage_partition.h"
|
| @@ -350,9 +350,8 @@ namespace {
|
| class ChromeContentBrowserClientClearSiteDataTest : public testing::Test {
|
| public:
|
| void SetUp() override {
|
| - BrowsingDataRemoverFactory::GetForBrowserContext(profile())
|
| - ->SetEmbedderDelegate(
|
| - base::MakeUnique<MockBrowsingDataRemoverDelegate>());
|
| + content::BrowserContext::GetBrowsingDataRemover(profile())
|
| + ->SetEmbedderDelegate(&mock_delegate_);
|
| run_loop_.reset(new base::RunLoop());
|
| }
|
|
|
| @@ -360,7 +359,7 @@ class ChromeContentBrowserClientClearSiteDataTest : public testing::Test {
|
|
|
| MockBrowsingDataRemoverDelegate* delegate() {
|
| return static_cast<MockBrowsingDataRemoverDelegate*>(
|
| - BrowsingDataRemoverFactory::GetForBrowserContext(profile())
|
| + content::BrowserContext::GetBrowsingDataRemover(profile())
|
| ->GetEmbedderDelegate());
|
| }
|
|
|
| @@ -373,6 +372,7 @@ class ChromeContentBrowserClientClearSiteDataTest : public testing::Test {
|
|
|
| private:
|
| std::unique_ptr<base::RunLoop> run_loop_;
|
| + MockBrowsingDataRemoverDelegate mock_delegate_;
|
| content::TestBrowserThreadBundle thread_bundle_;
|
| TestingProfile profile_;
|
| };
|
| @@ -392,30 +392,30 @@ TEST_F(ChromeContentBrowserClientClearSiteDataTest, Parameters) {
|
| } test_cases[] = {
|
| {false, false, false, 0},
|
| {true, false, false,
|
| - BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| - BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| + content::BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| + content::BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| ChromeBrowsingDataRemoverDelegate::DATA_TYPE_PLUGIN_DATA},
|
| - {false, true, false, BrowsingDataRemover::DATA_TYPE_DOM_STORAGE},
|
| - {false, false, true, BrowsingDataRemover::DATA_TYPE_CACHE},
|
| + {false, true, false, content::BrowsingDataRemover::DATA_TYPE_DOM_STORAGE},
|
| + {false, false, true, content::BrowsingDataRemover::DATA_TYPE_CACHE},
|
| {true, true, false,
|
| - BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| - BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| + content::BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| + content::BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| ChromeBrowsingDataRemoverDelegate::DATA_TYPE_PLUGIN_DATA |
|
| - BrowsingDataRemover::DATA_TYPE_DOM_STORAGE},
|
| + content::BrowsingDataRemover::DATA_TYPE_DOM_STORAGE},
|
| {true, false, true,
|
| - BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| - BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| + content::BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| + content::BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| ChromeBrowsingDataRemoverDelegate::DATA_TYPE_PLUGIN_DATA |
|
| - BrowsingDataRemover::DATA_TYPE_CACHE},
|
| + content::BrowsingDataRemover::DATA_TYPE_CACHE},
|
| {false, true, true,
|
| - BrowsingDataRemover::DATA_TYPE_DOM_STORAGE |
|
| - BrowsingDataRemover::DATA_TYPE_CACHE},
|
| + content::BrowsingDataRemover::DATA_TYPE_DOM_STORAGE |
|
| + content::BrowsingDataRemover::DATA_TYPE_CACHE},
|
| {true, true, true,
|
| - BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| - BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| + content::BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| + content::BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| ChromeBrowsingDataRemoverDelegate::DATA_TYPE_PLUGIN_DATA |
|
| - BrowsingDataRemover::DATA_TYPE_DOM_STORAGE |
|
| - BrowsingDataRemover::DATA_TYPE_CACHE},
|
| + content::BrowsingDataRemover::DATA_TYPE_DOM_STORAGE |
|
| + content::BrowsingDataRemover::DATA_TYPE_CACHE},
|
| };
|
|
|
| for (unsigned int i = 0; i < arraysize(test_cases); ++i) {
|
| @@ -430,8 +430,8 @@ TEST_F(ChromeContentBrowserClientClearSiteDataTest, Parameters) {
|
| // calls. In the latter case, the removal mask will be split into two
|
| // parts - one for the origin deletion and one for the registrable domain.
|
| const int domain_scoped_types =
|
| - BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| - BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| + content::BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| + content::BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| ChromeBrowsingDataRemoverDelegate::DATA_TYPE_PLUGIN_DATA;
|
| int registrable_domain_deletion_mask = test_case.mask & domain_scoped_types;
|
| int origin_deletion_mask = test_case.mask & ~domain_scoped_types;
|
| @@ -513,8 +513,8 @@ TEST_F(ChromeContentBrowserClientClearSiteDataTest, RegistrableDomains) {
|
|
|
| delegate()->ExpectCall(
|
| base::Time(), base::Time::Max(),
|
| - BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| - BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| + content::BrowsingDataRemover::DATA_TYPE_COOKIES |
|
| + content::BrowsingDataRemover::DATA_TYPE_CHANNEL_IDS |
|
| ChromeBrowsingDataRemoverDelegate::DATA_TYPE_PLUGIN_DATA,
|
| ChromeBrowsingDataRemoverDelegate::ALL_ORIGIN_TYPES,
|
| *registrable_domain_filter_builder);
|
| @@ -525,7 +525,7 @@ TEST_F(ChromeContentBrowserClientClearSiteDataTest, RegistrableDomains) {
|
| origin_filter_builder->AddOrigin(url::Origin(GURL(test_case.origin)));
|
|
|
| delegate()->ExpectCall(base::Time(), base::Time::Max(),
|
| - BrowsingDataRemover::DATA_TYPE_CACHE,
|
| + content::BrowsingDataRemover::DATA_TYPE_CACHE,
|
| ChromeBrowsingDataRemoverDelegate::ALL_ORIGIN_TYPES,
|
| *origin_filter_builder);
|
|
|
|
|