Pass all info to account chooser infobar.

components/autofill/core/common/password_form.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_AUTOFILL_CORE_COMMON_PASSWORD_FORM_H__
 #define COMPONENTS_AUTOFILL_CORE_COMMON_PASSWORD_FORM_H__
 
 #include <map>
 #include <string>
 #include <vector>
 
+#if defined(OS_ANDROID)
+#include "base/android/jni_array.h"
+#include "base/android/scoped_java_ref.h"
+#endif
 #include "base/time/time.h"
 #include "components/autofill/core/common/form_data.h"
 #include "url/gurl.h"
 
 namespace autofill {
 
 // The PasswordForm struct encapsulates information about a login form,
 // which can be an HTML form or a dialog with username/password text fields.
 //
 // The Web Data database stores saved username/passwords and associated form
@@ skipping 216 matching lines @@
   // Once user selects this credential the flag is reseted.
   bool skip_zero_click;
 
   // Returns true if this match was found using public suffix matching.
   bool IsPublicSuffixMatch() const;
 
   // Equality operators for testing.
   bool operator==(const PasswordForm& form) const;
   bool operator!=(const PasswordForm& form) const;
 
+#if defined(OS_ANDROID)
+  base::android::ScopedJavaLocalRef<jobject> createNativeCredential(JNIEnv* env,

[vabr (Chromium), 2015/02/26 09:29:32]

Note that PasswordForm is a struct, so all its data members are public. That
means it is easy to write non-member functions to use those data members,
without the need to add them as methods.

Because of the added functions being specific to Android, I suggest moving them
out of PasswordForm, and putting them somewhere near the code using these
functions. In any case, they should go to some android directory, into files
only compiled for Android. That would get rid of the #ifdefs, and keep
PasswordForm platform independent.

+                                                                    int type);
+
+  // Creates Java counterpart of PasswordForm, assigning it a |position| in case
+  // form is part of some array of forms and |type| which should be either local
+  // or federated.
+  base::android::ScopedJavaLocalRef<jobject>
+  createNativeCredential(JNIEnv* env, int position, int type);
+#endif
+
   PasswordForm();
   ~PasswordForm();
 };
 
 // Map username to PasswordForm* for convenience. See password_form_manager.h.
 typedef std::map<base::string16, PasswordForm*> PasswordFormMap;
 
 typedef std::map<base::string16, const PasswordForm*> ConstPasswordFormMap;
 
 // For testing.
 std::ostream& operator<<(std::ostream& os,
                          const autofill::PasswordForm& form);
+#if defined(OS_ANDROID)
+base::android::ScopedJavaLocalRef<jobjectArray> CreateNativeCredentialArray(
+    JNIEnv* env,
+    size_t size);
+
+bool RegisterCredential(JNIEnv* env);
+#endif
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CORE_COMMON_PASSWORD_FORM_H__