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

Unified Diff: chrome/browser/password_manager/password_store_mac.cc

Issue 2320763002: Support origin-based deletion for password store statistics (Closed)
Patch Set: Mac-specific code 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
Index: chrome/browser/password_manager/password_store_mac.cc
diff --git a/chrome/browser/password_manager/password_store_mac.cc b/chrome/browser/password_manager/password_store_mac.cc
index 391bb3508592de3aa79177b43bbb4cc96f072661..fdda11d51e56f38ef92ba53d88e84bd88860c76e 100644
--- a/chrome/browser/password_manager/password_store_mac.cc
+++ b/chrome/browser/password_manager/password_store_mac.cc
@@ -1243,12 +1243,13 @@ PasswordStoreChangeList PasswordStoreMac::DisableAutoSignInForOriginsImpl(
return changes;
}
-bool PasswordStoreMac::RemoveStatisticsCreatedBetweenImpl(
+bool PasswordStoreMac::RemoveStatisticsByOriginAndTimeImpl(
+ const base::Callback<bool(const GURL&)>& origin_filter,
base::Time delete_begin,
base::Time delete_end) {
return login_metadata_db_ &&
- login_metadata_db_->stats_table().RemoveStatsBetween(delete_begin,
- delete_end);
+ login_metadata_db_->stats_table().RemoveStatsByOriginAndTime(
+ origin_filter, delete_begin, delete_end);
}
std::vector<std::unique_ptr<PasswordForm>> PasswordStoreMac::FillMatchingLogins(
« no previous file with comments | « chrome/browser/password_manager/password_store_mac.h ('k') | chrome/browser/password_manager/password_store_proxy_mac.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698