a11y: Exclude children of nested keyboard accessible controls from a11y tree.

ui/views/accessibility/native_view_accessibility_mac.mm

Index: ui/views/accessibility/native_view_accessibility_mac.mm
diff --git a/ui/views/accessibility/native_view_accessibility_mac.mm b/ui/views/accessibility/native_view_accessibility_mac.mm
index 27b1a4fc8cbba80ac0db3ba3c142a45b76d70a29..ce500585fbba0874d7f0a2c7343057012f56c92b 100644
--- a/ui/views/accessibility/native_view_accessibility_mac.mm
+++ b/ui/views/accessibility/native_view_accessibility_mac.mm
@@ -18,8 +18,13 @@ NativeViewAccessibilityMac::NativeViewAccessibilityMac(View* view)
     : NativeViewAccessibility(view) {}
 
 gfx::NativeViewAccessible NativeViewAccessibilityMac::GetParent() {
-  if (view_->parent())
-    return view_->parent()->GetNativeViewAccessible();
+  View* parent_view = view_->parent();
+  if (parent_view) {
+    if (NativeViewAccessibility::GetForView(parent_view)->GetData().role ==
+        ui::AX_ROLE_IGNORED)

[Patti Lor, 2017/02/21 03:29:17]

I just inlined IsViewA11yIgnored() from native_view_accessibility.cc here,
hopefully that's OK.

[tapted, 2017/02/21 06:01:12]

Can this do something like

gfx::NativeViewAccessible toolkit_parent =
NativeViewAccessibility::GetToolkitParent();
if (toolkit_parent)
  return toolkit_parent;

if (view_->GetWidget())
  return view_->GetWidget()->GetNativeView();

if (parent_widget_)
  return parent_widget_->GetRootView()->GetNativeViewAccessible();


(maybe this should be in that other CL...)

[Patti Lor, 2017/02/27 05:31:04]

Done, kinda - traded your GetToolkitParent() suggestion for a virtual method
that retrieves the Widget parent, since both will be adding a new method anyway.

+      return GetForView(parent_view)->GetParent();
+    return parent_view->GetNativeViewAccessible();
+  }
 
   if (view_->GetWidget())
     return view_->GetWidget()->GetNativeView();