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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/ui/passwords/manage_passwords_ui_controller_unittest.cc

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
« no previous file with comments | « chrome/browser/ui/passwords/manage_passwords_ui_controller.cc ('k') | chrome/browser/ui/passwords/passwords_client_ui_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/passwords/manage_passwords_ui_controller_unittest.cc
diff --git a/chrome/browser/ui/passwords/manage_passwords_ui_controller_unittest.cc b/chrome/browser/ui/passwords/manage_passwords_ui_controller_unittest.cc
index a48451fa81fc7a218862d05f0e54f6be4f2d5ae6..2bf5bbf3a9b9b7f7cec6aac66c4b98132a7b933f 100644
--- a/chrome/browser/ui/passwords/manage_passwords_ui_controller_unittest.cc
+++ b/chrome/browser/ui/passwords/manage_passwords_ui_controller_unittest.cc
@@ -180,12 +180,12 @@
void ExpectIconStateIs(password_manager::ui::State state);
void ExpectIconAndControllerStateIs(password_manager::ui::State state);
- scoped_refptr<password_manager::PasswordFormManager>
+ std::unique_ptr<password_manager::PasswordFormManager>
CreateFormManagerWithBestMatches(
const autofill::PasswordForm& observed_form,
const std::vector<const autofill::PasswordForm*>& best_matches);
- scoped_refptr<password_manager::PasswordFormManager> CreateFormManager();
+ std::unique_ptr<password_manager::PasswordFormManager> CreateFormManager();
// Tests that the state is not changed when the password is autofilled.
void TestNotChangingStateOnAutofill(
@@ -241,11 +241,11 @@
EXPECT_EQ(state, controller()->GetState());
}
-scoped_refptr<password_manager::PasswordFormManager>
+std::unique_ptr<password_manager::PasswordFormManager>
ManagePasswordsUIControllerTest::CreateFormManagerWithBestMatches(
const autofill::PasswordForm& observed_form,
const std::vector<const autofill::PasswordForm*>& best_matches) {
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
new password_manager::PasswordFormManager(
&password_manager_, &client_, driver_.AsWeakPtr(), observed_form,
base::WrapUnique(new password_manager::StubFormSaver), &fetcher_));
@@ -253,7 +253,7 @@
return test_form_manager;
}
-scoped_refptr<password_manager::PasswordFormManager>
+std::unique_ptr<password_manager::PasswordFormManager>
ManagePasswordsUIControllerTest::CreateFormManager() {
return CreateFormManagerWithBestMatches(test_local_form(),
{&test_local_form()});
@@ -266,7 +266,7 @@
state == password_manager::ui::CONFIRMATION_STATE);
// Set the bubble state to |state|.
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
test_form_manager->ProvisionallySave(
test_local_form(),
@@ -318,7 +318,7 @@
}
TEST_F(ManagePasswordsUIControllerTest, PasswordSubmitted) {
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
test_form_manager->ProvisionallySave(
test_local_form(),
@@ -338,7 +338,7 @@
blacklisted.origin = test_local_form().origin;
blacklisted.signon_realm = blacklisted.origin.spec();
blacklisted.blacklisted_by_user = true;
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager =
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager =
CreateFormManagerWithBestMatches(test_local_form(), {&blacklisted});
EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());
controller()->OnPasswordSubmitted(std::move(test_form_manager));
@@ -351,7 +351,7 @@
TEST_F(ManagePasswordsUIControllerTest, PasswordSubmittedBubbleSuppressed) {
CreateSmartBubbleFieldTrial();
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
std::vector<password_manager::InteractionsStats> stats(1);
stats[0].origin_domain = test_local_form().origin.GetOrigin();
@@ -375,7 +375,7 @@
TEST_F(ManagePasswordsUIControllerTest, PasswordSubmittedBubbleNotSuppressed) {
CreateSmartBubbleFieldTrial();
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
std::vector<password_manager::InteractionsStats> stats(1);
stats[0].origin_domain = test_local_form().origin.GetOrigin();
@@ -397,7 +397,7 @@
}
TEST_F(ManagePasswordsUIControllerTest, PasswordSaved) {
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
test_form_manager->ProvisionallySave(
test_local_form(),
@@ -410,7 +410,7 @@
}
TEST_F(ManagePasswordsUIControllerTest, PasswordBlacklisted) {
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
test_form_manager->ProvisionallySave(
test_local_form(),
@@ -423,7 +423,7 @@
}
TEST_F(ManagePasswordsUIControllerTest, NormalNavigations) {
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());
controller()->OnPasswordSubmitted(std::move(test_form_manager));
@@ -442,7 +442,7 @@
}
TEST_F(ManagePasswordsUIControllerTest, NormalNavigationsClosedBubble) {
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());
controller()->OnPasswordSubmitted(std::move(test_form_manager));
@@ -467,7 +467,7 @@
content::WebContentsTester::For(web_contents())
->NavigateAndCommit(GURL("chrome://sign-in"));
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
test_form_manager->ProvisionallySave(
test_local_form(),
@@ -500,7 +500,7 @@
}
TEST_F(ManagePasswordsUIControllerTest, AutomaticPasswordSave) {
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());
controller()->OnAutomaticPasswordSave(std::move(test_form_manager));
@@ -744,7 +744,7 @@
}
TEST_F(ManagePasswordsUIControllerTest, UpdatePasswordSubmitted) {
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());
controller()->OnUpdatePasswordSubmitted(std::move(test_form_manager));
@@ -755,7 +755,7 @@
}
TEST_F(ManagePasswordsUIControllerTest, PasswordUpdated) {
- scoped_refptr<password_manager::PasswordFormManager> test_form_manager(
+ std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
CreateFormManager());
test_form_manager->ProvisionallySave(
test_local_form(),
« no previous file with comments | « chrome/browser/ui/passwords/manage_passwords_ui_controller.cc ('k') | chrome/browser/ui/passwords/passwords_client_ui_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698