View API/implementation cleanup:...

ui/views/focus/focus_manager.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/views/focus/focus_manager.h"
 
 #include <algorithm>
 
 #include "build/build_config.h"
 
@@ skipping 125 matching lines @@
 
   // Intercept arrow key messages to switch between grouped views.
   // TODO(beng): Perhaps make this a FocusTraversable that is created for
   //             Views that have a group set?
   ui::KeyboardCode key_code = event.key_code();
   if (focused_view_ && focused_view_->group() != -1 &&
       (key_code == ui::VKEY_UP || key_code == ui::VKEY_DOWN ||
        key_code == ui::VKEY_LEFT || key_code == ui::VKEY_RIGHT)) {
     bool next = (key_code == ui::VKEY_RIGHT || key_code == ui::VKEY_DOWN);
     std::vector<View*> views;
-    focused_view_->parent()->GetViewsWithGroup(focused_view_->group(), &views);
+    focused_view_->parent()->GetViewsInGroup(focused_view_->group(), &views);
     std::vector<View*>::const_iterator iter = std::find(views.begin(),
                                                         views.end(),
                                                         focused_view_);
     DCHECK(iter != views.end());
     int index = static_cast<int>(iter - views.begin());
     index += next ? 1 : -1;
     if (index < 0) {
       index = static_cast<int>(views.size()) - 1;
     } else if (index >= static_cast<int>(views.size())) {
       index = 0;
@@ skipping 307 matching lines @@
         !IsTraverseForward(direction),
         FocusSearch::DOWN,
         false,
         &new_focus_traversable,
         &new_starting_view);
   }
   return v;
 }
 
 }  // namespace ui