Credentials chooser UI for Android.

chrome/browser/ui/passwords/manage_passwords_ui_controller.cc

Index: chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
diff --git a/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc b/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
index 7f44b422f408ba9b5ac4573354f48545f3d53e69..7577c6703849f893065a6c855d21d1604f222c3f 100644
--- a/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
+++ b/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
@@ -24,6 +24,7 @@
 
 #if defined(OS_ANDROID)
 #include "chrome/browser/android/chromium_application.h"
+#include "chrome/browser/password_manager/account_chooser_infobar_delegate_android.h"
 #endif
 
 using autofill::PasswordFormMap;
@@ -132,7 +133,12 @@ bool ManagePasswordsUIController::OnChooseCredentials(
   // The map is useless because usernames may overlap.
   password_form_map_.clear();
   SetState(password_manager::ui::CREDENTIAL_REQUEST_AND_BUBBLE_STATE);
+#if defined(OS_ANDROID)
+  AccountChooserInfoBarDelegateAndroid::Create(web_contents(), this);
+  bubble_shown_ = true;

[vabr (Chromium), 2015/02/04 08:43:08]

This seems ad-hoc. It makes the comment at the declaration of bubble_shown_ no
longer true. There already is a !ANDROID #ifdef in
UpdateBubbleAndIconVisibility(), should this code be moved to an #else branch of
that one?

[melandory, 2015/02/04 15:58:50]

Done.

+#else
   UpdateBubbleAndIconVisibility();
+#endif
   if (bubble_shown_)
     credentials_callback_ = callback;
   return bubble_shown_;