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

Unified Diff: chrome/browser/password_manager/chrome_password_manager_client.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: chrome/browser/password_manager/chrome_password_manager_client.cc
diff --git a/chrome/browser/password_manager/chrome_password_manager_client.cc b/chrome/browser/password_manager/chrome_password_manager_client.cc
index b6c0edbde69cc91e9d1d77476679cc4163a0a78d..19c012e16377d5760ab30a6c29960c8b85e8ede8 100644
--- a/chrome/browser/password_manager/chrome_password_manager_client.cc
+++ b/chrome/browser/password_manager/chrome_password_manager_client.cc
@@ -254,7 +254,7 @@ bool ChromePasswordManagerClient::OnCredentialManagerUsed() {
}
bool ChromePasswordManagerClient::PromptUserToSaveOrUpdatePassword(
- std::unique_ptr<password_manager::PasswordFormManager> form_to_save,
+ scoped_refptr<password_manager::PasswordFormManager> form_to_save,
bool update_password) {
// Save password infobar and the password bubble prompts in case of
// "webby" URLs and do not prompt in case of "non-webby" URLS (e.g. file://).
@@ -374,7 +374,7 @@ void ChromePasswordManagerClient::NotifyStorePasswordCalled() {
}
void ChromePasswordManagerClient::AutomaticPasswordSave(
- std::unique_ptr<password_manager::PasswordFormManager> saved_form) {
+ scoped_refptr<password_manager::PasswordFormManager> saved_form) {
#if defined(OS_ANDROID)
GeneratedPasswordSavedInfoBarDelegateAndroid::Create(web_contents());
#else

Powered by Google App Engine
This is Rietveld 408576698