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

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

Issue 2132063002: Implement origin-based deletion for password manager's auto-signin bit. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed PasswordStoreMac Created 4 years, 5 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 1ffb69d04692c1b1c8f90ffea44624e2ddcc92bf..111c8613ffd9e3bd593e4f79afc40d8f0ca38b70 100644
--- a/components/password_manager/core/browser/password_store.cc
+++ b/components/password_manager/core/browser/password_store.cc
@@ -149,10 +149,12 @@ void PasswordStore::RemoveStatisticsCreatedBetween(
delete_begin, delete_end, completion));
}
-void PasswordStore::DisableAutoSignInForAllLogins(
+void PasswordStore::DisableAutoSignInForOrigins(
+ const base::Callback<bool(const GURL&)>& origin_filter,
const base::Closure& completion) {
- ScheduleTask(base::Bind(&PasswordStore::DisableAutoSignInForAllLoginsInternal,
- this, completion));
+ ScheduleTask(
+ base::Bind(&PasswordStore::DisableAutoSignInForOriginsInternal, this,
+ base::Callback<bool(const GURL&)>(origin_filter), completion));
}
void PasswordStore::TrimAffiliationCache() {
@@ -417,9 +419,10 @@ void PasswordStore::RemoveStatisticsCreatedBetweenInternal(
main_thread_runner_->PostTask(FROM_HERE, completion);
}
-void PasswordStore::DisableAutoSignInForAllLoginsInternal(
+void PasswordStore::DisableAutoSignInForOriginsInternal(
+ const base::Callback<bool(const GURL&)>& origin_filter,
const base::Closure& completion) {
- DisableAutoSignInForAllLoginsImpl();
+ DisableAutoSignInForOriginsImpl(origin_filter);
if (!completion.is_null())
main_thread_runner_->PostTask(FROM_HERE, completion);
}

Powered by Google App Engine
This is Rietveld 408576698