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

Unified Diff: ui/views/focus/focus_search.h

Issue 10949005: Fix toolbar keyboard accessibility on Views (alternative impl). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix linux_chromeos compile Created 8 years, 3 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: ui/views/focus/focus_search.h
diff --git a/ui/views/focus/focus_search.h b/ui/views/focus/focus_search.h
index 345a6cc4d60a49c2abe12bea2da0885cc94cad12..ab53d5f2b48db4f5dd10745512d4b2fd980481a0 100644
--- a/ui/views/focus/focus_search.h
+++ b/ui/views/focus/focus_search.h
@@ -68,6 +68,18 @@ class VIEWS_EXPORT FocusSearch {
FocusTraversable** focus_traversable,
View** focus_traversable_view);
+ protected:
+ // Get the parent, but stay within the root. Returns NULL if asked for
+ // the parent of root_. Subclasses can override this if they need custom
sky 2012/09/18 21:46:06 |root_|
dmazzoni 2012/09/19 00:23:40 Done.
+ // focus search behavior.
+ virtual View* GetParent(View* v);
+
+ // Returns true if |v| is contained within the hierarchy rooted at |root|.
+ // Subclasses can override this if they need custom focus search behavior.
+ virtual bool Contains(View* root, const View* v);
+
+ View* root() const { return root_; }
+
private:
// Convenience method that returns true if a view is focusable and does not
// belong to the specified group.
@@ -82,10 +94,6 @@ class VIEWS_EXPORT FocusSearch {
// specified view is returned.
View* FindSelectedViewForGroup(View* view);
- // Get the parent, but stay within the root. Returns NULL if asked for
- // the parent of root_.
- View* GetParent(View* view);
-
// Returns the next focusable view or view containing a FocusTraversable
// (NULL if none was found), starting at the starting_view.
// |check_starting_view|, |can_go_up| and |can_go_down| controls the

Powered by Google App Engine
This is Rietveld 408576698