Index: chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc |
diff --git a/chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc b/chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc |
index ba7587ba88977e9230016d9eb86a614674c695ae..45b3f69fabf3b5af5d6b5e58a0c9fc673a2e9734 100644 |
--- a/chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc |
+++ b/chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc |
@@ -43,37 +43,26 @@ class ManagePasswordsBubbleModelTest : public testing::Test { |
void PretendPasswordWaiting() { |
model_->set_state(password_manager::ui::PENDING_PASSWORD_STATE); |
model_->OnBubbleShown(ManagePasswordsBubble::AUTOMATIC); |
- controller()->SetState( |
- password_manager::ui::PENDING_PASSWORD_STATE); |
} |
void PretendCredentialsWaiting() { |
model_->set_state(password_manager::ui::CREDENTIAL_REQUEST_STATE); |
model_->OnBubbleShown(ManagePasswordsBubble::AUTOMATIC); |
- controller()->SetState(password_manager::ui::CREDENTIAL_REQUEST_STATE); |
} |
void PretendAutoSigningIn() { |
model_->set_state(password_manager::ui::AUTO_SIGNIN_STATE); |
model_->OnBubbleShown(ManagePasswordsBubble::AUTOMATIC); |
- controller()->SetState(password_manager::ui::AUTO_SIGNIN_STATE); |
} |
void PretendManagingPasswords() { |
model_->set_state(password_manager::ui::MANAGE_STATE); |
model_->OnBubbleShown(ManagePasswordsBubble::USER_ACTION); |
- controller()->SetState(password_manager::ui::MANAGE_STATE); |
} |
void PretendBlacklisted() { |
model_->set_state(password_manager::ui::BLACKLIST_STATE); |
model_->OnBubbleShown(ManagePasswordsBubble::USER_ACTION); |
- |
- base::string16 kTestUsername = base::ASCIIToUTF16("test_username"); |
- autofill::ConstPasswordFormMap map; |
- map[kTestUsername] = &test_form_; |
- controller()->SetPasswordFormMap(map); |
- controller()->SetState(password_manager::ui::BLACKLIST_STATE); |
} |
ManagePasswordsUIControllerMock* controller() { |
@@ -227,6 +216,7 @@ TEST_F(ManagePasswordsBubbleModelTest, ClickUnblacklist) { |
EXPECT_EQ(password_manager::ui::MANAGE_STATE, model_->state()); |
EXPECT_FALSE(controller()->saved_password()); |
EXPECT_FALSE(controller()->never_saved_password()); |
+ EXPECT_TRUE(controller()->unblacklist_site()); |
histogram_tester.ExpectUniqueSample( |
kUIDismissalReasonMetric, |
@@ -264,7 +254,7 @@ TEST_F(ManagePasswordsBubbleModelTest, ClickCancelCredential) { |
password_manager::metrics_util::CLICKED_NOPE); |
EXPECT_FALSE(controller()->saved_password()); |
EXPECT_FALSE(controller()->never_saved_password()); |
- EXPECT_TRUE(controller()->choose_credential()); |
+ EXPECT_FALSE(controller()->choose_credential()); |
histogram_tester.ExpectUniqueSample( |
kUIDismissalReasonMetric, |
@@ -281,7 +271,7 @@ TEST_F(ManagePasswordsBubbleModelTest, DismissCredential) { |
password_manager::metrics_util::NO_DIRECT_INTERACTION); |
EXPECT_FALSE(controller()->saved_password()); |
EXPECT_FALSE(controller()->never_saved_password()); |
- EXPECT_TRUE(controller()->choose_credential()); |
+ EXPECT_FALSE(controller()->choose_credential()); |
histogram_tester.ExpectUniqueSample( |
kUIDismissalReasonMetric, |