Improve Android accessible hit testing.

content/browser/accessibility/browser_accessibility_android.h

Index: content/browser/accessibility/browser_accessibility_android.h
diff --git a/content/browser/accessibility/browser_accessibility_android.h b/content/browser/accessibility/browser_accessibility_android.h
index 09681c439324da6170ad5e8dc91e064313c37a0c..7498abfd66b345da1e3b627ac60a5c89cde54871 100644
--- a/content/browser/accessibility/browser_accessibility_android.h
+++ b/content/browser/accessibility/browser_accessibility_android.h
@@ -48,6 +48,16 @@ class CONTENT_EXPORT BrowserAccessibilityAndroid : public BrowserAccessibility {
   bool IsSlider() const;
   bool IsVisibleToUser() const;
 
+  // This returns true for all nodes that we should navigate to.
+  // Nodes that have a generic role, no accessible name, and aren't
+  // focusable or clickable aren't interesting.
+  bool IsInterestingOnAndroid() const;
+
+  // If this node isn't interesting, recursively checks all of the

[aboxhall, 2016/10/18 22:17:28]

This comment is missing a critical point, especially given the name gives the
same misleading idea: it's not strictly a descendant, it may be |this|.

[dmazzoni, 2016/10/19 23:00:45]

Ah, good point.

Renamed and reworded, take a look.

+  // platform children of this node, and if just a single one is
+  // interesting, returns that one.
+  const BrowserAccessibilityAndroid* GetSoleInterestingDescendant() const;
+
   bool CanOpenPopup() const;
 
   bool HasFocusableChild() const;