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

Unified Diff: chrome/browser/browsing_data/browsing_data_remover.cc

Issue 2084903002: Moved BrowsingDataCounter and part of BrowsingDataCounterUtils to components. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed deps Created 4 years, 6 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_remover.cc
diff --git a/chrome/browser/browsing_data/browsing_data_remover.cc b/chrome/browser/browsing_data/browsing_data_remover.cc
index c58ab51a4923239b8d8d0f3ea16a5e224ae75b50..99a65d6688a091cab03636c89f16d2c8ac11b842 100644
--- a/chrome/browser/browsing_data/browsing_data_remover.cc
+++ b/chrome/browser/browsing_data/browsing_data_remover.cc
@@ -290,37 +290,6 @@ BrowsingDataRemover::NotificationDetails::NotificationDetails(
BrowsingDataRemover::NotificationDetails::~NotificationDetails() {}
-// static
-BrowsingDataRemover::TimeRange BrowsingDataRemover::Unbounded() {
- return TimeRange(base::Time(), base::Time::Max());
-}
-
-// static
-BrowsingDataRemover::TimeRange BrowsingDataRemover::Period(TimePeriod period) {
- switch (period) {
- case LAST_HOUR:
- content::RecordAction(
- UserMetricsAction("ClearBrowsingData_LastHour"));
- break;
- case LAST_DAY:
- content::RecordAction(
- UserMetricsAction("ClearBrowsingData_LastDay"));
- break;
- case LAST_WEEK:
- content::RecordAction(
- UserMetricsAction("ClearBrowsingData_LastWeek"));
- break;
- case FOUR_WEEKS:
- content::RecordAction(
- UserMetricsAction("ClearBrowsingData_LastMonth"));
- break;
- case EVERYTHING:
- content::RecordAction(
- UserMetricsAction("ClearBrowsingData_Everything"));
- break;
- }
- return TimeRange(CalculateBeginDeleteTime(period), base::Time::Max());
-}
BrowsingDataRemover::BrowsingDataRemover(
content::BrowserContext* browser_context)
@@ -1077,30 +1046,6 @@ void BrowsingDataRemover::ClearSettingsForOneTypeWithPredicate(
}
}
-base::Time BrowsingDataRemover::CalculateBeginDeleteTime(
- TimePeriod time_period) {
- base::TimeDelta diff;
- base::Time delete_begin_time = base::Time::Now();
- switch (time_period) {
- case LAST_HOUR:
- diff = base::TimeDelta::FromHours(1);
- break;
- case LAST_DAY:
- diff = base::TimeDelta::FromHours(24);
- break;
- case LAST_WEEK:
- diff = base::TimeDelta::FromHours(7*24);
- break;
- case FOUR_WEEKS:
- diff = base::TimeDelta::FromHours(4*7*24);
- break;
- case EVERYTHING:
- delete_begin_time = base::Time();
- break;
- }
- return delete_begin_time - diff;
-}
-
bool BrowsingDataRemover::AllDone() {
return !waiting_for_clear_autofill_origin_urls_ &&
!waiting_for_clear_cache_ && !waiting_for_clear_content_licenses_ &&

Powered by Google App Engine
This is Rietveld 408576698