Don't post events on internal nodes in the accessibility tree on Android.

content/browser/accessibility/browser_accessibility_manager_android.cc

Index: content/browser/accessibility/browser_accessibility_manager_android.cc
diff --git a/content/browser/accessibility/browser_accessibility_manager_android.cc b/content/browser/accessibility/browser_accessibility_manager_android.cc
index df7361110a178fe0e03c4bd458df4bbaeacbbb1d..9a36bc51229c0be31188c855f1d0765a4f46fd7d 100644
--- a/content/browser/accessibility/browser_accessibility_manager_android.cc
+++ b/content/browser/accessibility/browser_accessibility_manager_android.cc
@@ -206,9 +206,6 @@ void BrowserAccessibilityManagerAndroid::NotifyAccessibilityEvent(
   if (obj.is_null())
     return;
 
-  BrowserAccessibilityAndroid* android_node =
-      static_cast<BrowserAccessibilityAndroid*>(node);
-
   if (event_type == ui::AX_EVENT_HIDE)
     return;
 
@@ -225,6 +222,16 @@ void BrowserAccessibilityManagerAndroid::NotifyAccessibilityEvent(
     return;
   }
 
+  // Sometimes we get events on nodes in our internal accessibility tree
+  // that aren't exposed on Android. Walk up the ancestors and update |node|
+  // to point to the highest ancestor that's a leaf node.
+  BrowserAccessibility* parent = node->GetParent();
+  while (parent) {
+    if (parent->PlatformIsLeaf())
+      node = parent;

[David Tseng, 2016/10/04 19:56:15]

Break? Are there leaves that have ancestor leaves?

[dmazzoni, 2016/10/04 21:35:09]

In theory yes? I want this to behave as if you walked the tree.

+    parent = parent->GetParent();
+  }
+
   // Always send AccessibilityEvent.TYPE_WINDOW_CONTENT_CHANGED to notify
   // the Android system that the accessibility hierarchy rooted at this
   // node has changed.
@@ -237,6 +244,8 @@ void BrowserAccessibilityManagerAndroid::NotifyAccessibilityEvent(
     return;
   }
 
+  BrowserAccessibilityAndroid* android_node =
+      static_cast<BrowserAccessibilityAndroid*>(node);
   switch (event_type) {
     case ui::AX_EVENT_LOAD_COMPLETE:
       Java_BrowserAccessibilityManager_handlePageLoaded(