Credential Manager API: pop up the new "Manage accounts" bubble.

components/autofill/core/common/password_form.h

Index: components/autofill/core/common/password_form.h
diff --git a/components/autofill/core/common/password_form.h b/components/autofill/core/common/password_form.h
index 783eda54786c8e433d20807d7799e65c5523b62b..31e3ed05352bb0021b76182b27d8631270e647f9 100644
--- a/components/autofill/core/common/password_form.h
+++ b/components/autofill/core/common/password_form.h
@@ -254,6 +254,11 @@ typedef std::map<base::string16, PasswordForm*> PasswordFormMap;
 
 typedef std::map<base::string16, const PasswordForm*> ConstPasswordFormMap;
 
+// True if the unique key for the forms is the same. The unique key is

[Ilya Sherman, 2015/02/24 21:45:48]

nit: "key for the forms is the same" -> "keys for the forms are the same"
("keys", "are")

[vasilii, 2015/02/25 09:39:50]

Done.

+// (origin, username_element, username_value, password_element, signon_realm).
+// It's used by Sync and LoginDatabase.

[Ilya Sherman, 2015/02/24 21:45:48]

I don't see these uses in the diff.  In fact, I only see one use, within
//chrome/browser/ui/passwords/manage_passwords_ui_controller.cc.  Does this
method need to be defined in the password_form header and implementation at all?

[vasilii, 2015/02/25 09:39:50]

I move this function temporarily out of here. It will be used by
native_backend_kwallet_x.cc in the future.

+bool IsEqualUniqueKey(const PasswordForm& left, const PasswordForm& right);
+
 // For testing.
 std::ostream& operator<<(std::ostream& os,
                          const autofill::PasswordForm& form);