DialogClientView: Fix regression in Chrome Task Manager focusing.

ui/views/window/dialog_client_view.h

 // Copyright (c) 2012 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 UI_VIEWS_WINDOW_DIALOG_CLIENT_VIEW_H_
 #define UI_VIEWS_WINDOW_DIALOG_CLIENT_VIEW_H_
 
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
 #include "ui/base/ui_base_types.h"
@@ skipping 66 matching lines @@
   // View implementation.
   void ChildPreferredSizeChanged(View* child) override;
   void ChildVisibilityChanged(View* child) override;
 
  private:
   bool has_dialog_buttons() const { return ok_button_ || cancel_button_; }
 
   // Create a dialog button of the appropriate type.
   LabelButton* CreateDialogButton(ui::DialogButton type);
 
+  // Adds |ok_button| as a child of this view at the appropriate index, to
+  // ensure correct focus order.
+  void AddOKButton(LabelButton* ok_button);
+
+  // Adds |cancel_button| as a child of this view at the appropriate index, to
+  // ensure correct focus order.
+  void AddCancelButton(LabelButton* cancel_button);
+
   // Update |button|'s text and enabled state according to the delegate's state.
   void UpdateButton(LabelButton* button, ui::DialogButton type);
 
   // Returns the height of the row containing the buttons and the extra view.
   int GetButtonsAndExtraViewRowHeight() const;
 
   // Returns the insets for the buttons and extra view.
   gfx::Insets GetButtonRowInsets() const;
 
   // How much to inset the button row.
   gfx::Insets button_row_insets_;
 
-  // Sets up the focus chain for the child views. This is required since the
-  // delegate may choose to add/remove views at any time.
-  void SetupFocusChain();
-
   // The dialog buttons.
   LabelButton* ok_button_;
   LabelButton* cancel_button_;
 
   // The extra view shown in the row of buttons; may be NULL.
   View* extra_view_;
 
   // True if we've notified the delegate the window is closing and the delegate
   // allowed the close. In some situations it's possible to get two closes (see
   // http://crbug.com/71940). This is used to avoid notifying the delegate
   // twice, which can have bad consequences.
   bool delegate_allowed_close_;
 
   DISALLOW_COPY_AND_ASSIGN(DialogClientView);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_WINDOW_DIALOG_CLIENT_VIEW_H_