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

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

Issue 2900693002: [Password Manager] Convert |pending_login_managers_| to an array of scoped_refptr (Closed)
Patch Set: Rebase Created 3 years, 7 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/credential_manager_password_form_manager.cc
diff --git a/components/password_manager/core/browser/credential_manager_password_form_manager.cc b/components/password_manager/core/browser/credential_manager_password_form_manager.cc
index 4406d199befce93e1b68dcb05f56c52dc476a1d0..ffdeb11a5278c3f227e789a408ddfdf83f6a0f72 100644
--- a/components/password_manager/core/browser/credential_manager_password_form_manager.cc
+++ b/components/password_manager/core/browser/credential_manager_password_form_manager.cc
@@ -45,9 +45,6 @@ CredentialManagerPasswordFormManager::CredentialManagerPasswordFormManager(
GrabFetcher(std::move(form_fetcher));
}
-CredentialManagerPasswordFormManager::~CredentialManagerPasswordFormManager() {
-}
-
void CredentialManagerPasswordFormManager::ProcessMatches(
const std::vector<const PasswordForm*>& non_federated,
size_t filtered_count) {
@@ -68,6 +65,8 @@ void CredentialManagerPasswordFormManager::ProcessMatches(
weak_factory_.GetWeakPtr()));
}
+CredentialManagerPasswordFormManager::~CredentialManagerPasswordFormManager() {}
+
void CredentialManagerPasswordFormManager::NotifyDelegate() {
delegate_->OnProvisionalSaveComplete();
}

Powered by Google App Engine
This is Rietveld 408576698