From 330dc11b876d4b92852dd27fcdbc3cf30283ac85 Mon Sep 17 00:00:00 2001
From: "ben@chromium.org"
 <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Date: Thu, 3 Feb 2011 17:29:32 +0000
Subject: 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
---
 ui/views/focus/focus_search.cc | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

(limited to 'ui/views/focus/focus_search.cc')

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
-- 
cgit v1.1