Show user credentials chooser bubble.

chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.cc

Index: chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.cc
diff --git a/chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.cc b/chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.cc
index 103e5d616a54dee75ffc986e1874653a16f6bdb5..7773b49e5ab24650e7ccf59257a8e170903dd98c 100644
--- a/chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.cc
+++ b/chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.cc
@@ -5,13 +5,15 @@
 #include "chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.h"
 
 #include "content/public/browser/web_contents.h"
+#include "testing/gtest/include/gtest/gtest.h"
 
 ManagePasswordsUIControllerMock::ManagePasswordsUIControllerMock(
     content::WebContents* contents)
     : ManagePasswordsUIController(contents),
       navigated_to_settings_page_(false),
       saved_password_(false),
-      never_saved_password_(false) {
+      never_saved_password_(false),
+      choose_credential_(false) {
   contents->SetUserData(UserDataKey(), this);
 }
 
@@ -24,13 +26,18 @@ void ManagePasswordsUIControllerMock::
 }
 
 const autofill::PasswordForm&
-    ManagePasswordsUIControllerMock::PendingCredentials() const {
-  return pending_credentials_;
+    ManagePasswordsUIControllerMock::PendingPassword() const {
+  return pending_password_;
 }
 
-void ManagePasswordsUIControllerMock::SetPendingCredentials(
-    autofill::PasswordForm pending_credentials) {
-  pending_credentials_ = pending_credentials;
+void ManagePasswordsUIControllerMock::SetPendingPassword(
+    autofill::PasswordForm pending_password) {
+  pending_password_ = pending_password;
+}
+
+void ManagePasswordsUIControllerMock::UpdateBubbleAndIconVisibility() {
+  ManagePasswordsUIController::UpdateBubbleAndIconVisibility();
+  OnBubbleShown();

[Mike West, 2014/11/21 11:21:12]

Hrm. I'm surprised this doesn't change behavior in some way in the other tests.
Why do we need to pretend we showed the bubble every time we update the
visibility? Doesn't the actual method also close the bubble, or just leave it
alone, depending on the state?

[vasilii, 2014/11/21 16:45:37]

I call ManagePasswordsUIController::OnChooseCredentials in the tests. It returns
true only if the bubble was shown in UpdateBubbleAndIconVisibility().

 }
 
 bool ManagePasswordsUIControllerMock::IsInstalled() const {
@@ -44,3 +51,10 @@ void ManagePasswordsUIControllerMock::SavePasswordInternal() {
 void ManagePasswordsUIControllerMock::NeverSavePasswordInternal() {
   never_saved_password_ = true;
 }
+
+void ManagePasswordsUIControllerMock::ChooseCredential(
+    bool was_chosen,
+    const autofill::PasswordForm& form) {
+  EXPECT_FALSE(choose_credential_);
+  choose_credential_ = true;
+}