Credential Manager API: Showing both local and federated logins.

chrome/browser/ui/passwords/manage_passwords_ui_controller.h

Index: chrome/browser/ui/passwords/manage_passwords_ui_controller.h
diff --git a/chrome/browser/ui/passwords/manage_passwords_ui_controller.h b/chrome/browser/ui/passwords/manage_passwords_ui_controller.h
index 5a1838e20073936b25a9ff88a0fc35e7f22e60a7..71b65e72a0f173b561bffe120327d1f4a37b92c9 100644
--- a/chrome/browser/ui/passwords/manage_passwords_ui_controller.h
+++ b/chrome/browser/ui/passwords/manage_passwords_ui_controller.h
@@ -18,6 +18,7 @@ class WebContents;
 }
 
 namespace password_manager {
+enum class CredentialType : unsigned int;
 struct CredentialInfo;
 class PasswordFormManager;
 }
@@ -80,8 +81,9 @@ class ManagePasswordsUIController
 
   // Called from the model when the user chooses a credential.
   // The controller MUST be in a pending credentials state.
-  virtual void ChooseCredential(bool was_chosen,
-                                const autofill::PasswordForm& form);
+  virtual void ChooseCredential(
+      const autofill::PasswordForm& form,
+      password_manager::CredentialType credential_type);
 
   // Called from the model when the user chooses to never save passwords; passes
   // the action off to the FormManager. The controller MUST be in a pending
@@ -111,6 +113,14 @@ class ManagePasswordsUIController
     return new_password_forms_;
   }
 
+  ScopedVector<autofill::PasswordForm>& federated_credentials_forms() {
+    return federated_credentials_forms_;
+  }
+
+  ScopedVector<autofill::PasswordForm>& local_credentials_forms() {
+    return local_credentials_forms_;
+  }
+
   // True if a password is sitting around, waiting for a user to decide whether
   // or not to save it.
   bool PasswordPendingUserDecision() const;
@@ -154,6 +164,12 @@ class ManagePasswordsUIController
   // |password_form_map_| is to be cleared too.
   ScopedVector<autofill::PasswordForm> new_password_forms_;
 
+  // Federated credentials.

[vasilii, 2015/01/13 18:13:28]

The comment should explaine that they are used just in the specific state.

+  ScopedVector<autofill::PasswordForm> federated_credentials_forms_;
+
+  // Local credentials.
+  ScopedVector<autofill::PasswordForm> local_credentials_forms_;
+
   // All previously stored credentials for a specific site.
   // Protected, not private, so we can mess with the value in
   // ManagePasswordsUIControllerMock.