Removes unnecessary calls to FocusManager

ui/app_list/views/folder_header_view.cc

 // Copyright (c) 2013 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/app_list/views/folder_header_view.h"
 
 #include <algorithm>
 
 #include "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "ui/app_list/app_list_constants.h"
 #include "ui/app_list/app_list_folder_item.h"
 #include "ui/app_list/app_list_switches.h"
 #include "ui/app_list/resources/grit/app_list_resources.h"
 #include "ui/app_list/views/app_list_folder_view.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/canvas.h"
 #include "ui/strings/grit/ui_strings.h"
 #include "ui/views/border.h"
 #include "ui/views/controls/button/image_button.h"
 #include "ui/views/controls/textfield/textfield.h"
-#include "ui/views/focus/focus_manager.h"
 #include "ui/views/painter.h"
 
 namespace app_list {
 
 namespace {
 
 const int kPreferredWidth = 360;
 const int kPreferredHeight = 48;
 const int kBottomSeparatorHeight = 1;
 const int kMaxFolderNameWidth = 300;
@@ skipping 56 matching lines @@
   folder_name_visible_ = visible;
   Update();
   SchedulePaint();
 }
 
 void FolderHeaderView::OnFolderItemRemoved() {
   folder_item_ = NULL;
 }
 
 void FolderHeaderView::SetTextFocus() {
-  if (!folder_name_view_->HasFocus()) {
-    views::FocusManager* focus_manager = GetFocusManager();
-    if (focus_manager)
-      focus_manager->SetFocusedView(folder_name_view_);
-  }
+  folder_name_view_->RequestFocus();

[sky, 2017/03/10 15:45:03]

There's no need to check if the view has focus. Focus code checks it already.

[msw, 2017/03/10 17:47:08]

Acknowledged.

 }
 
 bool FolderHeaderView::HasTextFocus() const {
   return folder_name_view_->HasFocus();
 }
 
 void FolderHeaderView::Update() {
   if (!folder_item_)
     return;
 
@@ skipping 97 matching lines @@
 void FolderHeaderView::ButtonPressed(views::Button* sender,
                                      const ui::Event& event) {
   delegate_->NavigateBack(folder_item_, event);
 }
 
 void FolderHeaderView::ItemNameChanged() {
   Update();
 }
 
 }  // namespace app_list