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

Unified Diff: components/password_manager/core/browser/password_store.cc

Issue 1072413003: Add foundation for trimming the AffiliationDatabase. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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: components/password_manager/core/browser/password_store.cc
diff --git a/components/password_manager/core/browser/password_store.cc b/components/password_manager/core/browser/password_store.cc
index 5b13a3cb86e2f294dc3ac95555d326abdf738cae..c242ecb3845fdb109a4b1fde503db7a2bebf4c95 100644
--- a/components/password_manager/core/browser/password_store.cc
+++ b/components/password_manager/core/browser/password_store.cc
@@ -97,10 +97,12 @@ void PasswordStore::RemoveLogin(const PasswordForm& form) {
ScheduleTask(base::Bind(&PasswordStore::RemoveLoginInternal, this, form));
}
-void PasswordStore::RemoveLoginsCreatedBetween(base::Time delete_begin,
- base::Time delete_end) {
+void PasswordStore::RemoveLoginsCreatedBetween(
+ base::Time delete_begin,
+ base::Time delete_end,
+ const base::Closure& completion) {
ScheduleTask(base::Bind(&PasswordStore::RemoveLoginsCreatedBetweenInternal,
- this, delete_begin, delete_end));
+ this, delete_begin, delete_end, completion));
}
void PasswordStore::RemoveLoginsSyncedBetween(base::Time delete_begin,
@@ -109,6 +111,11 @@ void PasswordStore::RemoveLoginsSyncedBetween(base::Time delete_begin,
this, delete_begin, delete_end));
}
+void PasswordStore::TrimAffiliationCache() {
+ if (affiliated_match_helper_)
vasilii 2015/04/23 13:16:35 Should it be a DCHECK()?
+ affiliated_match_helper_->TrimAffiliationCache();
+}
+
void PasswordStore::GetLogins(const PasswordForm& form,
AuthorizationPromptPolicy prompt_policy,
PasswordStoreConsumer* consumer) {
@@ -288,11 +295,15 @@ void PasswordStore::RemoveLoginInternal(const PasswordForm& form) {
NotifyLoginsChanged(changes);
}
-void PasswordStore::RemoveLoginsCreatedBetweenInternal(base::Time delete_begin,
- base::Time delete_end) {
+void PasswordStore::RemoveLoginsCreatedBetweenInternal(
+ base::Time delete_begin,
+ base::Time delete_end,
+ const base::Closure& completion) {
PasswordStoreChangeList changes =
RemoveLoginsCreatedBetweenImpl(delete_begin, delete_end);
NotifyLoginsChanged(changes);
+ if (!completion.is_null())
+ main_thread_runner_->PostTask(FROM_HERE, completion);
}
void PasswordStore::RemoveLoginsSyncedBetweenInternal(base::Time delete_begin,

Powered by Google App Engine
This is Rietveld 408576698