Android Autofill Accessibility, Phase I

content/browser/accessibility/browser_accessibility_manager_android.cc

 // 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.
 
 #include "content/browser/accessibility/browser_accessibility_manager_android.h"
 
 #include <stddef.h>
 
 #include <cmath>
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
+#include "base/feature_list.h"
 #include "base/i18n/char_iterator.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
+#include "content/app/strings/grit/content_strings.h"
 #include "content/browser/accessibility/browser_accessibility_android.h"
 #include "content/browser/accessibility/one_shot_accessibility_tree_search.h"
 #include "content/common/accessibility_messages.h"
+#include "content/public/common/content_features.h"
 #include "jni/BrowserAccessibilityManager_jni.h"
 #include "ui/accessibility/ax_text_utils.h"
 
 using base::android::AttachCurrentThread;
 using base::android::JavaParamRef;
 using base::android::ScopedJavaLocalRef;
 
 namespace content {
 
 namespace {
@@ skipping 80 matching lines @@
   InitSearchKeyToPredicateMapIfNeeded();
   const auto& iter = g_search_key_to_predicate_map.Get().find(element_type);
   if (iter != g_search_key_to_predicate_map.Get().end())
     return iter->second;
 
   // If we don't recognize the selector, return any element that a
   // screen reader should navigate to.
   return AllInterestingNodesPredicate;
 }
 
+// The element in the document for which we may be displaying an autofill popup.
+int32_t g_element_hosting_autofill_popup_id_ = -1;

[dmazzoni, 2017/03/29 17:28:43]

Nit: no training underscore on globals, only on class members.

[csashi, 2017/03/29 18:26:27]

Done.

+
+// Autofill popup will not be part of the |AXTree| that is sent by renderer.
+// Hence, we need a proxy |AXNode| to represent the autofill popup.
+BrowserAccessibility* g_autofill_popup_proxy_node_ = nullptr;
+ui::AXNode* g_autofill_popup_proxy_node_ax_node_ = nullptr;
+
+void DeleteAutofillPopupProxy() {
+  if (g_autofill_popup_proxy_node_) {
+    g_autofill_popup_proxy_node_->Destroy();
+    delete g_autofill_popup_proxy_node_ax_node_;
+    g_autofill_popup_proxy_node_ = nullptr;
+  }
+}
+
 }  // anonymous namespace
 
 namespace aria_strings {
   const char kAriaLivePolite[] = "polite";
   const char kAriaLiveAssertive[] = "assertive";
 }
 
 // static
 BrowserAccessibilityManager* BrowserAccessibilityManager::Create(
     const ui::AXTreeUpdate& initial_tree,
@@ skipping 18 matching lines @@
   Initialize(initial_tree);
   SetContentViewCore(content_view_core);
 }
 
 BrowserAccessibilityManagerAndroid::~BrowserAccessibilityManagerAndroid() {
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> obj = GetJavaRefFromRootManager();
   if (obj.is_null())
     return;
 
+  // Clean up autofill popup proxy node in case the popup was not dismissed.
+  DeleteAutofillPopupProxy();
+
   Java_BrowserAccessibilityManager_onNativeObjectDestroyed(
       env, obj, reinterpret_cast<intptr_t>(this));
 }
 
 // static
 ui::AXTreeUpdate
     BrowserAccessibilityManagerAndroid::GetEmptyDocument() {
   ui::AXNodeData empty_document;
   empty_document.id = 0;
   empty_document.role = ui::AX_ROLE_ROOT_WEB_AREA;
@@ skipping 568 matching lines @@
         env, obj, android_node->unique_id());
   }
 }
 
 jint BrowserAccessibilityManagerAndroid::FindElementType(
     JNIEnv* env,
     const JavaParamRef<jobject>& obj,
     jint start_id,
     const JavaParamRef<jstring>& element_type_str,
     jboolean forwards) {
+  // Navigate forwards to the autofill popup's proxy node if focus is currently
+  // on the element hosting the autofill popup. Once within the popup, a back
+  // press will navigate back to the element hosting the popup.
+  if (forwards && start_id == g_element_hosting_autofill_popup_id_ &&
+      g_autofill_popup_proxy_node_) {
+    return g_autofill_popup_proxy_node_->unique_id();
+  }
+
   BrowserAccessibilityAndroid* start_node = GetFromUniqueID(start_id);
   if (!start_node)
     return 0;
 
   BrowserAccessibilityManager* root_manager = GetRootManager();
   if (!root_manager)
     return 0;
 
   BrowserAccessibility* root = root_manager->GetRoot();
   if (!root)
@@ skipping 155 matching lines @@
   }
 
   return true;
 }
 
 void BrowserAccessibilityManagerAndroid::SetAccessibilityFocus(
     JNIEnv* env,
     const JavaParamRef<jobject>& obj,
     jint id) {
   BrowserAccessibilityAndroid* node = GetFromUniqueID(id);
-  if (!node)
-    return;
-
-  node->manager()->SetAccessibilityFocus(*node);
+  if (node)
+    node->manager()->SetAccessibilityFocus(*node);
 }
 
 bool BrowserAccessibilityManagerAndroid::IsSlider(
     JNIEnv* env,
     const JavaParamRef<jobject>& obj,
     jint id) {
   BrowserAccessibilityAndroid* node = GetFromUniqueID(id);
   if (!node)
     return false;
 
   return node->GetRole() == ui::AX_ROLE_SLIDER;
 }
 
+void BrowserAccessibilityManagerAndroid::OnAutofillPopupDisplayed(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
+  if (!base::FeatureList::IsEnabled(features::kAndroidAutofillAccessibility))
+    return;
+
+  BrowserAccessibility* current_focus = GetFocus();
+  if (current_focus == nullptr) {
+    return;
+  }
+
+  DeleteAutofillPopupProxy();
+
+  g_autofill_popup_proxy_node_ = BrowserAccessibility::Create();
+  g_autofill_popup_proxy_node_ax_node_ = new ui::AXNode(nullptr, -1, -1);
+  ui::AXNodeData ax_node_data;
+  ax_node_data.role = ui::AX_ROLE_MENU;
+  ax_node_data.SetName("Autofill");
+  ax_node_data.state = 1 << ui::AX_STATE_READ_ONLY;
+  ax_node_data.state |= 1 << ui::AX_STATE_FOCUSABLE;
+  ax_node_data.state |= 1 << ui::AX_STATE_SELECTABLE;
+  g_autofill_popup_proxy_node_ax_node_->SetData(ax_node_data);
+  g_autofill_popup_proxy_node_->Init(this,
+                                     g_autofill_popup_proxy_node_ax_node_);
+
+  g_element_hosting_autofill_popup_id_ = current_focus->unique_id();

[dmazzoni, 2017/03/29 17:28:43]

Rename g_element_hosting_autofill_popup_id_
g_element_hosting_autofill_popup_unique_id - we have multiple types of ids, and
"unique id" should always be named such to avoid confusion

[csashi, 2017/03/29 18:26:27]

Done.

+}
+
+void BrowserAccessibilityManagerAndroid::OnAutofillPopupDismissed(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
+  g_element_hosting_autofill_popup_id_ = -1;
+  DeleteAutofillPopupProxy();
+}
+
+jboolean BrowserAccessibilityManagerAndroid::IsAutofillPopupNode(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj,
+    jint id) {
+  return g_autofill_popup_proxy_node_ &&
+         g_autofill_popup_proxy_node_->unique_id() == id;
+}
+
 bool BrowserAccessibilityManagerAndroid::Scroll(
     JNIEnv* env,
     const JavaParamRef<jobject>& obj,
     jint id,
     int direction) {
   BrowserAccessibilityAndroid* node = GetFromUniqueID(id);
   if (!node)
     return false;
 
   return node->Scroll(direction);
@@ skipping 38 matching lines @@
 
   JNIEnv* env = AttachCurrentThread();
   return root_manager->java_ref().get(env);
 }
 
 bool RegisterBrowserAccessibilityManager(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace content