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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 2320763002: Support origin-based deletion for password store statistics (Closed)
Patch Set: Created 4 years, 3 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
« no previous file with comments | « no previous file | components/password_manager/core/browser/mock_password_store.h » ('j') | components/password_manager/core/browser/password_store.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 1c6d65a5e81bb797c35eda4da85035fe70741a26..dbd29e5eb2023b8bf821ae63e7aa6ebe8724171c 100644
--- a/chrome/browser/browsing_data/browsing_data_remover.cc
+++ b/chrome/browser/browsing_data/browsing_data_remover.cc
@@ -905,8 +905,8 @@ void BrowsingDataRemover::RemoveImpl(
if (password_store) {
waiting_for_clear_passwords_stats_ = true;
- password_store->RemoveStatisticsCreatedBetween(
- delete_begin_, delete_end_,
+ password_store->RemoveStatisticsByOriginAndTime(
+ nullable_filter, delete_begin_, delete_end_,
base::Bind(&BrowsingDataRemover::OnClearedPasswordsStats,
weak_ptr_factory_.GetWeakPtr()));
}
« no previous file with comments | « no previous file | components/password_manager/core/browser/mock_password_store.h » ('j') | components/password_manager/core/browser/password_store.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698