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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/ui/webui/options/clear_browser_data_handler.cc

Issue 2578723002: Reduce BrowsingDataRemover's dependencies on Chrome (Closed)
Patch Set: Created 4 years 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
« chrome/browser/browsing_data/browsing_data_remover_unittest.cc ('K') | « chrome/browser/ui/webui/net_internals/net_internals_ui.cc ('k') | chrome/browser/ui/webui/settings/settings_clear_browsing_data_handler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/options/clear_browser_data_handler.cc
diff --git a/chrome/browser/ui/webui/options/clear_browser_data_handler.cc b/chrome/browser/ui/webui/options/clear_browser_data_handler.cc
index b12d01e42ab1d93bd2d4d86f2a6491b0dd7786bd..cce1050044ad491b26ede19a2c1fc51862f3ed93 100644
--- a/chrome/browser/ui/webui/options/clear_browser_data_handler.cc
+++ b/chrome/browser/ui/webui/options/clear_browser_data_handler.cc
@@ -319,9 +319,12 @@ void ClearBrowserDataHandler::HandleClearBrowserData(
remover_->AddObserver(this);
int period_selected =
prefs->GetInteger(browsing_data::prefs::kDeleteTimePeriod);
+ browsing_data::TimePeriod time_period =
+ static_cast<browsing_data::TimePeriod>(period_selected);
+ browsing_data::RecordDeletionForPeriod(time_period);
remover_->RemoveAndReply(
- BrowsingDataRemover::Period(
- static_cast<browsing_data::TimePeriod>(period_selected)),
+ browsing_data::CalculateBeginDeleteTime(time_period),
+ browsing_data::CalculateEndDeleteTime(time_period),
remove_mask, origin_mask, this);
// Store the clear browsing data time. Next time the clear browsing data
« chrome/browser/browsing_data/browsing_data_remover_unittest.cc ('K') | « chrome/browser/ui/webui/net_internals/net_internals_ui.cc ('k') | chrome/browser/ui/webui/settings/settings_clear_browsing_data_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698