Android Autofill Accessibility, Phase I

content/browser/accessibility/browser_accessibility_manager_android.h

Index: content/browser/accessibility/browser_accessibility_manager_android.h
diff --git a/content/browser/accessibility/browser_accessibility_manager_android.h b/content/browser/accessibility/browser_accessibility_manager_android.h
index e5110dc823a54e8be6cf2c0f460d98c8c3a35e5a..637ea2046f7693df405b4b17dc3d5d7d2ed4d584 100644
--- a/content/browser/accessibility/browser_accessibility_manager_android.h
+++ b/content/browser/accessibility/browser_accessibility_manager_android.h
@@ -99,6 +99,7 @@ class CONTENT_EXPORT BrowserAccessibilityManagerAndroid
   jboolean IsNodeValid(JNIEnv* env,
                        const base::android::JavaParamRef<jobject>& obj,
                        jint id);
+
   void HitTest(JNIEnv* env,
                const base::android::JavaParamRef<jobject>& obj,
                jint x,
@@ -222,6 +223,20 @@ class CONTENT_EXPORT BrowserAccessibilityManagerAndroid
                 const base::android::JavaParamRef<jobject>& obj,
                 jint id);
 
+  // Accessibility methods to support navigation for autofill popup.
+  void OnAutofillPopupDisplayed(
+      JNIEnv* env,
+      const base::android::JavaParamRef<jobject>& obj);
+  void OnAutofillPopupDismissed(
+      JNIEnv* env,
+      const base::android::JavaParamRef<jobject>& obj);

[dmazzoni, 2017/03/28 16:47:35]

Do we need both IsAutofillPopupNode and
GetElementHostingAutofillPopupId?

[csashi, 2017/03/28 17:48:10]

Done.

+  jboolean IsAutofillPopupNode(JNIEnv* env,
+                               const base::android::JavaParamRef<jobject>& obj,
+                               jint id);
+  jint GetElementHostingAutofillPopupId(
+      JNIEnv* env,
+      const base::android::JavaParamRef<jobject>& obj);
+
   // Scrolls any scrollable container by about 80% of one page in the
   // given direction.
   bool Scroll(JNIEnv* env,