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

Unified Diff: components/password_manager/core/browser/password_manager.h

Issue 2926833002: Revert of [Password Manager] Convert |pending_login_managers_| to an array of scoped_refptr (Closed)
Patch Set: Created 3 years, 6 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_manager.h
diff --git a/components/password_manager/core/browser/password_manager.h b/components/password_manager/core/browser/password_manager.h
index 30990f6982fb4a533298628e5d9e0c7fcfef3691..b85165d98ed64cd788c37762ad08ba0c6bc2e488 100644
--- a/components/password_manager/core/browser/password_manager.h
+++ b/components/password_manager/core/browser/password_manager.h
@@ -13,7 +13,6 @@
#include "base/callback.h"
#include "base/gtest_prod_util.h"
#include "base/macros.h"
-#include "base/memory/ref_counted.h"
#include "base/observer_list.h"
#include "base/strings/string16.h"
#include "build/build_config.h"
@@ -185,7 +184,7 @@
#if defined(UNIT_TEST)
// TODO(crbug.com/639786): Replace using this by quering the factory for
// mocked PasswordFormManagers.
- const std::vector<scoped_refptr<PasswordFormManager>>&
+ const std::vector<std::unique_ptr<PasswordFormManager>>&
pending_login_managers() {
return pending_login_managers_;
}
@@ -267,7 +266,7 @@
// When a form is "seen" on a page, a PasswordFormManager is created
// and stored in this collection until user navigates away from page.
- std::vector<scoped_refptr<PasswordFormManager>> pending_login_managers_;
+ std::vector<std::unique_ptr<PasswordFormManager>> pending_login_managers_;
// When the user submits a password/credential, this contains the
// PasswordFormManager for the form in question until we deem the login
@@ -275,7 +274,7 @@
// send the PasswordFormManager back to the pending_login_managers_ set.
// Scoped in case PasswordManager gets deleted (e.g tab closes) between the
// time a user submits a login form and gets to the next page.
- scoped_refptr<PasswordFormManager> provisional_save_manager_;
+ std::unique_ptr<PasswordFormManager> provisional_save_manager_;
// The embedder-level client. Must outlive this class.
PasswordManagerClient* const client_;

Powered by Google App Engine
This is Rietveld 408576698