Credential Manager: Return the first valid item from the PasswordStore.

components/password_manager/content/common/credential_manager_types.h

Index: components/password_manager/content/common/credential_manager_types.h
diff --git a/components/password_manager/content/common/credential_manager_types.h b/components/password_manager/content/common/credential_manager_types.h
index 68242c92dd4800bc63fc9af4935e45b72f8668b6..c825e6f8e6dcbfa025236c0625d1aad294d3b590 100644
--- a/components/password_manager/content/common/credential_manager_types.h
+++ b/components/password_manager/content/common/credential_manager_types.h
@@ -12,6 +12,10 @@
 #include "base/strings/string16.h"
 #include "url/gurl.h"
 
+namespace autofill {
+struct PasswordForm;
+}
+
 namespace password_manager {
 
 // Limit the size of the federations array that we pass to the browser to
@@ -30,6 +34,7 @@ struct CredentialInfo {
   CredentialInfo(const base::string16& id,
                  const base::string16& name,
                  const GURL& avatar);
+  CredentialInfo(const autofill::PasswordForm& form);

[vabr (Chromium), 2014/09/30 12:16:55]

[No action required] Just out of curiosity, do you expect to use all three
constructors in the future, or is this exploratory, and likely to stick with 1
or 2?

   ~CredentialInfo();
 
   CredentialType type;