Show user credentials chooser bubble.

components/password_manager/core/common/password_manager_ui.cc

Index: components/password_manager/core/common/password_manager_ui.cc
diff --git a/components/password_manager/core/common/password_manager_ui.cc b/components/password_manager/core/common/password_manager_ui.cc
index b76394ed7f1a79662850d375e45ab29fd1a63903..84fe742ca60c494a26e5b2f5045660e12746d232 100644
--- a/components/password_manager/core/common/password_manager_ui.cc
+++ b/components/password_manager/core/common/password_manager_ui.cc
@@ -16,7 +16,8 @@ bool IsPendingState(State state) {
 
 bool IsAutomaticDisplayState(State state) {
   return state == PENDING_PASSWORD_AND_BUBBLE_STATE ||
-         state == CONFIRMATION_STATE;
+         state == CONFIRMATION_STATE ||
+         state == PENDING_CREDENTIALS_AND_BUBBLE_STATE;
 }
 
 State GetEndStateForAutomaticState(State state) {
@@ -24,10 +25,10 @@ State GetEndStateForAutomaticState(State state) {
   switch (state) {
     case PENDING_PASSWORD_AND_BUBBLE_STATE:
       return PENDING_PASSWORD_STATE;
-      break;
     case CONFIRMATION_STATE:
       return MANAGE_STATE;
-      break;
+    case PENDING_CREDENTIALS_AND_BUBBLE_STATE:
+      return PENDING_CREDENTIALS_AND_BUBBLE_STATE;

[Mike West, 2014/11/19 19:56:37]

This doesn't seem right. Shouldn't it move to MANAGE_STATE?

[vasilii, 2014/11/20 15:08:06]

Honestly, the code here has vague meaning to me. We change the state explicitly
in other places (see ManagePasswordsUIController::SavePassword for example).

We can move to MANAGE_STATE. But what passwords do we demonstrate there?

     default:
       NOTREACHED();
       return INACTIVE_STATE;