Fix reversed focus traversal order issue.

views/widget/root_view.h

 // Copyright (c) 2009 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.
 
 #ifndef VIEWS_WIDGET_ROOT_VIEW_H_
 #define VIEWS_WIDGET_ROOT_VIEW_H_
 
 #include <string>
 
 #include "base/ref_counted.h"
@@ skipping 126 matching lines @@
   // MouseWheel events are sent on the focus path.
   virtual bool ProcessMouseWheelEvent(const MouseWheelEvent& e);
 
   // Overridden to handle special root view case.
   virtual bool IsVisibleInRootView() const;
 
   // FocusTraversable implementation.
   virtual View* FindNextFocusableView(View* starting_view,
                                       bool reverse,
                                       Direction direction,
-                                      bool dont_loop,
+                                      bool check_starting_view,
                                       FocusTraversable** focus_traversable,
                                       View** focus_traversable_view);
   virtual FocusTraversable* GetFocusTraversableParent();
   virtual View* GetFocusTraversableParentView();
 
   // Used to set the FocusTraversable parent after the view has been created
   // (typically when the hierarchy changes and this RootView is added/removed).
   virtual void SetFocusTraversableParent(FocusTraversable* focus_traversable);
 
   // Used to set the View parent after the view has been created.
@@ skipping 49 matching lines @@
   // notifies the appropriate views.
   void ViewBoundsChanged(View* view, bool size_changed, bool position_changed);
 
   // Registers a view for notification when the visible bounds relative to the
   // root of a view changes.
   void RegisterViewForVisibleBoundsNotification(View* view);
   void UnregisterViewForVisibleBoundsNotification(View* view);
 
   // Returns the next focusable view or view containing a FocusTraversable (NULL
   // if none was found), starting at the starting_view.
-  // skip_starting_view, can_go_up and can_go_down controls the traversal of
+  // check_starting_view, can_go_up and can_go_down controls the traversal of
   // the views hierarchy.
   // skip_group_id specifies a group_id, -1 means no group. All views from a
   // group are traversed in one pass.
   View* FindNextFocusableViewImpl(View* starting_view,
-                                  bool skip_starting_view,
+                                  bool check_starting_view,
                                   bool can_go_up,
                                   bool can_go_down,
-                                  int skip_group_id);
+                                  int skip_group_id,
+                                  FocusTraversable** focus_traversable,
+                                  View** focus_traversable_view);
 
   // Same as FindNextFocusableViewImpl but returns the previous focusable view.
   View* FindPreviousFocusableViewImpl(View* starting_view,
-                                      bool skip_starting_view,
+                                      bool check_starting_view,
                                       bool can_go_up,
                                       bool can_go_down,
-                                      int skip_group_id);
+                                      int skip_group_id,
+                                      FocusTraversable** focus_traversable,
+                                      View** focus_traversable_view);
 
   // Convenience method that returns true if a view is focusable and does not
   // belong to the specified group.
   bool IsViewFocusableCandidate(View* v, int skip_group_id);
 
   // Returns the view selected for the group of the selected view. If the view
   // does not belong to a group or if no view is selected in the group, the
   // specified view is returned.
   static View* FindSelectedViewForGroup(View* view);
 
@@ skipping 92 matching lines @@
 
 #ifndef NDEBUG
   // True if we're currently processing paint.
   bool is_processing_paint_;
 #endif
 };
 
 }  // namespace views
 
 #endif // VIEWS_WIDGET_ROOT_VIEW_H_