Removes unnecessary calls to FocusManager

ui/app_list/views/folder_header_view.cc

Index: ui/app_list/views/folder_header_view.cc
diff --git a/ui/app_list/views/folder_header_view.cc b/ui/app_list/views/folder_header_view.cc
index c1488249d2bc17a2815937ac989d3e0b445ecf74..4ce264181a4a4fdd5f5eaeb34b7eb8fe9fc71f54 100644
--- a/ui/app_list/views/folder_header_view.cc
+++ b/ui/app_list/views/folder_header_view.cc
@@ -19,7 +19,6 @@
 #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 {
@@ -96,11 +95,7 @@ void FolderHeaderView::OnFolderItemRemoved() {
 }
 
 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 {