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

Unified Diff: chrome/browser/chrome_content_browser_client.cc

Issue 2578723002: Reduce BrowsingDataRemover's dependencies on Chrome (Closed)
Patch Set: A new callsite appeared through rebase - fixed the compilation error. Created 3 years, 12 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/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index 365f80caf02360491c4c9320b205db06e2900af9..0ff95daef242c15693adae8fab366567b3201b2c 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -121,6 +121,7 @@
#include "chromeos/chromeos_constants.h"
#include "components/autofill/content/browser/content_autofill_driver_factory.h"
#include "components/autofill/core/common/autofill_switches.h"
+#include "components/browsing_data/core/browsing_data_utils.h"
#include "components/cdm/browser/cdm_message_filter_android.h"
#include "components/cloud_devices/common/cloud_devices_switches.h"
#include "components/content_settings/core/browser/content_settings_utils.h"
@@ -2582,7 +2583,7 @@ void ChromeContentBrowserClient::ClearCache(RenderFrameHost* rfh) {
rfh->GetSiteInstance()->GetProcess()->GetBrowserContext());
BrowsingDataRemover* remover =
BrowsingDataRemoverFactory::GetForBrowserContext(profile);
- remover->Remove(BrowsingDataRemover::Unbounded(),
+ remover->Remove(base::Time(), base::Time::Max(),
BrowsingDataRemover::REMOVE_CACHE,
BrowsingDataHelper::UNPROTECTED_WEB);
}
@@ -2593,7 +2594,7 @@ void ChromeContentBrowserClient::ClearCookies(RenderFrameHost* rfh) {
BrowsingDataRemover* remover =
BrowsingDataRemoverFactory::GetForBrowserContext(profile);
int remove_mask = BrowsingDataRemover::REMOVE_SITE_DATA;
- remover->Remove(BrowsingDataRemover::Unbounded(), remove_mask,
+ remover->Remove(base::Time(), base::Time::Max(), remove_mask,
BrowsingDataHelper::UNPROTECTED_WEB);
}
@@ -2631,7 +2632,7 @@ void ChromeContentBrowserClient::ClearSiteData(
domain_filter_builder->AddRegisterableDomain(domain);
remover->RemoveWithFilterAndReply(
- BrowsingDataRemover::Period(browsing_data::TimePeriod::ALL_TIME),
+ base::Time(), base::Time::Max(),
BrowsingDataRemover::REMOVE_COOKIES |
BrowsingDataRemover::REMOVE_CHANNEL_IDS |
BrowsingDataRemover::REMOVE_PLUGIN_DATA,
@@ -2658,7 +2659,7 @@ void ChromeContentBrowserClient::ClearSiteData(
origin_filter_builder->AddOrigin(origin);
remover->RemoveWithFilterAndReply(
- BrowsingDataRemover::Period(browsing_data::TimePeriod::ALL_TIME),
+ base::Time(), base::Time::Max(),
remove_mask, BrowsingDataHelper::ALL, std::move(origin_filter_builder),
observer);
} else {

Powered by Google App Engine
This is Rietveld 408576698