diff options
author | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-03 17:29:32 +0000 |
---|---|---|
committer | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-03 17:29:32 +0000 |
commit | 330dc11b876d4b92852dd27fcdbc3cf30283ac85 (patch) | |
tree | ce4e27b9db9bdd0562b0b296de9078f1c05c9104 /ui/views/focus/focus_search.cc | |
parent | a33721a77396c8f7607d330cd3d07b82f4ebc0ed (diff) | |
download | chromium_src-330dc11b876d4b92852dd27fcdbc3cf30283ac85.zip chromium_src-330dc11b876d4b92852dd27fcdbc3cf30283ac85.tar.gz chromium_src-330dc11b876d4b92852dd27fcdbc3cf30283ac85.tar.bz2 |
Complete the Focus Manager hookup.
- Temporarily patch in old Focus Sibling initialization code.
- Improve the constness of FocusSearch
- Simplify ContainsView method in FocusManager
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6349080
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73632 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ui/views/focus/focus_search.cc')
-rw-r--r-- | ui/views/focus/focus_search.cc | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/ui/views/focus/focus_search.cc b/ui/views/focus/focus_search.cc index 2050b1d..c8093fd0 100644 --- a/ui/views/focus/focus_search.cc +++ b/ui/views/focus/focus_search.cc @@ -20,7 +20,7 @@ View* FocusSearch::FindNextFocusableView(View* starting_view, Direction direction, bool check_starting_view, FocusTraversable** focus_traversable, - View** focus_traversable_view) { + View** focus_traversable_view) const { *focus_traversable = NULL; *focus_traversable_view = NULL; @@ -94,20 +94,20 @@ View* FocusSearch::FindNextFocusableView(View* starting_view, return NULL; } -bool FocusSearch::IsViewFocusableCandidate(View* v, int skip_group_id) { +bool FocusSearch::IsViewFocusableCandidate(View* v, int skip_group_id) const { return IsFocusable(v) && (v->IsGroupFocusTraversable() || skip_group_id == -1 || v->group() != skip_group_id); } -bool FocusSearch::IsFocusable(View* v) { +bool FocusSearch::IsFocusable(View* v) const { if (accessibility_mode_) return v && v->IsAccessibilityFocusableInRootView(); return v && v->IsFocusableInRootView(); } -View* FocusSearch::FindSelectedViewForGroup(View* view) { +View* FocusSearch::FindSelectedViewForGroup(View* view) const { // No group for that view. if (view->IsGroupFocusTraversable() || view->group() == -1) return view; @@ -120,7 +120,7 @@ View* FocusSearch::FindSelectedViewForGroup(View* view) { return view; } -View* FocusSearch::GetParent(View* v) { +View* FocusSearch::GetParent(View* v) const { return root_->Contains(v) ? v->parent() : NULL; } @@ -139,7 +139,7 @@ View* FocusSearch::FindNextFocusableViewImpl( bool can_go_down, int skip_group_id, FocusTraversable** focus_traversable, - View** focus_traversable_view) { + View** focus_traversable_view) const { if (check_starting_view) { if (IsViewFocusableCandidate(starting_view, skip_group_id)) { View* v = FindSelectedViewForGroup(starting_view); @@ -212,7 +212,7 @@ View* FocusSearch::FindPreviousFocusableViewImpl( bool can_go_down, int skip_group_id, FocusTraversable** focus_traversable, - View** focus_traversable_view) { + View** focus_traversable_view) const { // Let's go down and right as much as we can. if (can_go_down) { // Before we go into the direct children, we have to check if this view has |