Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(161)

Unified Diff: content/browser/accessibility/browser_accessibility_android.h

Issue 2422773002: Improve Android accessible hit testing. (Closed)
Patch Set: Clarify that it can return |this| Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..6942bb65644dbd4318f40d062631590bb4e892a2 100644
--- a/content/browser/accessibility/browser_accessibility_android.h
+++ b/content/browser/accessibility/browser_accessibility_android.h
@@ -48,6 +48,18 @@ 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 is interesting (IsInterestingOnAndroid() returns true),
+ // returns |this|. If not, it recursively checks all of the
+ // platform children of this node, and if just a single one is
+ // interesting, returns that one. If no descendants are interesting, or
+ // if more than one is interesting, returns nullptr.
+ const BrowserAccessibilityAndroid* GetSoleInterestingNodeFromSubtree() const;
+
bool CanOpenPopup() const;
bool HasFocusableChild() const;

Powered by Google App Engine
This is Rietveld 408576698