WebView autofill implementation

components/autofill/android/autofill_provider_android.h

+// Copyright 2017 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_ANDROID_AUTOFILL_PROVIDER_ANDROID_H_
+#define COMPONENTS_AUTOFILL_ANDROID_AUTOFILL_PROVIDER_ANDROID_H_
+
+#include "base/android/jni_weak_ref.h"
+#include "base/memory/weak_ptr.h"
+#include "components/autofill/core/browser/autofill_provider.h"
+
+namespace content {
+class WebContents;
+}
+
+namespace autofill {
+
+class FormDataAndroid;
+
+class AutofillProviderAndroid : public AutofillProvider {

[Mathieu, 2017/05/24 18:02:01]

Is this only used for WebView? if so, perhaps if should be
WebViewAutofillProvider

[Mathieu, 2017/05/24 18:02:01]

Can you add a class comment, detailing who owns this and what its lifetime is
attached to (one per tab, singleton, etc.)

[michaelbai, 2017/05/24 20:57:54]

No, Chrome will also use this in future.

[michaelbai, 2017/05/26 23:12:07]

Done.

[Roger McFarlane (Chromium), 2017/05/31 18:33:23]

More details, plz. In what capacity?

[michaelbai, 2017/05/31 21:05:54]

I don't have detail to put, but will definitely involve your guys once we start
to work on it. If it turns out WebView only, I will move these files to
android_webview directory.

+ public:
+  AutofillProviderAndroid(const base::android::JavaRef<jobject>& jcaller,
+                          content::WebContents* web_contents);
+  ~AutofillProviderAndroid() override;
+
+  // Override AutofillProvider methods

[sebsg, 2017/05/24 13:32:42]

The usual syntax is

// AutofillProvider:


and then put all of those back to back without lines in between.

Thanks!

[michaelbai, 2017/05/26 23:12:07]

Done.

+  void OnQueryFormFieldAutofill(AutofillHandlerProxy* handler,
+                                int32_t id,
+                                const FormData& form,
+                                const FormFieldData& field,
+                                const gfx::RectF& bounding_box) override;
+

[Mathieu, 2017/05/24 18:02:01]

nit: no need for extra lines in this "override" section

[michaelbai, 2017/05/26 23:12:07]

Done.

+  void OnTextFieldDidChange(AutofillHandlerProxy* handler,
+                            const FormData& form,
+                            const FormFieldData& field,
+                            const base::TimeTicks& timestamp) override;
+
+  void OnWillSubmitForm(AutofillHandlerProxy* handler,
+                        const FormData& form,
+                        const base::TimeTicks& timestamp) override;
+
+  void OnFocusNoLongerOnForm(AutofillHandlerProxy* handler) override;
+
+  void OnDidFillAutofillFormData(AutofillHandlerProxy* handler,
+                                 const FormData& form,
+                                 base::TimeTicks timestamp) override;
+
+  void Reset(AutofillHandlerProxy* handler) override;
+
+  // Methods called by Java.
+  void OnAutofillAvailable(JNIEnv* env, jobject jcaller, jobject formData);

[Mathieu, 2017/05/24 18:02:01]

hacker case everywhere: form_data

[michaelbai, 2017/05/26 23:12:07]

Done.

+
+ private:
+  void OnFocusChanged(bool focus_on_form,
+                      size_t index,
+                      const gfx::RectF& bounding_box);
+
+  bool ValidateHandler(AutofillHandlerProxy* handler);
+
+  int32_t id_;
+  std::unique_ptr<FormDataAndroid> form_;
+  base::WeakPtr<AutofillHandlerProxy> handler_;
+  JavaObjectWeakGlobalRef java_ref_;
+  content::WebContents* web_contents_;
+
+  DISALLOW_COPY_AND_ASSIGN(AutofillProviderAndroid);
+};
+
+bool RegisterAutofillProvider(JNIEnv* env);
+
+}  // namespace autofill
+
+#endif  // COMPONENTS_AUTOFILL_ANDROID_AUTOFILL_PROVIDER_ANDROID_H_