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

ui/views/accessibility/native_view_accessibility.h

Index: ui/views/accessibility/native_view_accessibility.h
diff --git a/ui/views/accessibility/native_view_accessibility.h b/ui/views/accessibility/native_view_accessibility.h
index f4d61b3f87919fc25799dea2ae7c171ad2c9af63..abeedd54aead2eaa3748294590ec231a0f5d5858 100644
--- a/ui/views/accessibility/native_view_accessibility.h
+++ b/ui/views/accessibility/native_view_accessibility.h
@@ -49,7 +49,12 @@ class VIEWS_EXPORT NativeViewAccessibility
   // Focuses or unfocuses a View.
   bool SetFocused(bool focused);
 
-  // ui::AXPlatformNodeDelegate
+#if defined(OS_MACOSX)
+  // Checks if there are any keyboard focusable ancestors.
+  bool HasFocusableAncestor();

[tapted, 2016/11/25 04:36:27]

can this be moved to an anonymous namespace in the .cc only?

[Patti Lor, 2016/11/28 06:40:23]

Good idea, thanks! I actually realised that just using
IsAccessibilityFocusable() isn't good enough - doing that would mean the
children of disabled controls (which would be accessibility focusable if they
were enabled) would not be ignored. So I had to make View::focusable_behavior()
public, not sure if that's an OK change?

+#endif
+
+  // ui::AXPlatformNodeDelegate:
   const ui::AXNodeData& GetData() override;
   int GetChildCount() override;
   gfx::NativeViewAccessible ChildAtIndex(int index) override;
@@ -62,7 +67,7 @@ class VIEWS_EXPORT NativeViewAccessibility
   bool AccessibilityPerformAction(const ui::AXActionData& data) override;
   void DoDefaultAction() override;
 
-  // WidgetObserver
+  // WidgetObserver:
   void OnWidgetDestroying(Widget* widget) override;
 
   Widget* parent_widget() const { return parent_widget_; }