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

ui/accessibility/platform/ax_platform_node_mac.mm

 // Copyright 2014 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.
 
 #import "ui/accessibility/platform/ax_platform_node_mac.h"
 
 #import <Cocoa/Cocoa.h>
 #include <stddef.h>
 
 #include "base/macros.h"
@@ skipping 256 matching lines @@
 - (NSString*)getStringAttribute:(ui::AXStringAttribute)attribute {
   std::string attributeValue;
   if (node_->GetStringAttribute(attribute, &attributeValue))
     return base::SysUTF8ToNSString(attributeValue);
   return nil;
 }
 
 // NSAccessibility informal protocol implementation.
 
 - (BOOL)accessibilityIsIgnored {
-  return [[self AXRole] isEqualToString:NSAccessibilityUnknownRole];
+  return [[self AXRole] isEqualToString:NSAccessibilityUnknownRole] ||
+         node_->GetDelegate()->IsIgnored();
 }
 
 - (id)accessibilityHitTest:(NSPoint)point {
   for (AXPlatformNodeCocoa* child in [self AXChildren]) {
     if (NSPointInRect(point, child.boundsInScreen))

[tapted, 2016/10/18 04:38:31]

Perhaps we can tackle this logic. Something like

if (NSPointInRect(point, [child boundsInScreen]) && ![child
accessibilityIsIgnored])
  return [child ..


So we'll need the change just above still. However, a lot of the other changes
to native_view_accessibility.cc may become unnecessary.

I think this will still allow http://crbug.com/610589 to be fixed.

       return [child accessibilityHitTest:point];
   }
   return NSAccessibilityUnignoredAncestor(self);
 }
 
 - (NSArray*)accessibilityActionNames {
   return nil;
 }
 
 - (NSArray*)accessibilityAttributeNames {
@@ skipping 277 matching lines @@
   }
   NotifyMacEvent(target, event_type);
 }
 
 int AXPlatformNodeMac::GetIndexInParent() {
   // TODO(dmazzoni): implement this.  http://crbug.com/396137
   return -1;
 }
 
 }  // namespace ui