Do not give instant focus if a view's toplevelwidget is not active

chrome/test/BUILD.gn

Index: chrome/test/BUILD.gn
diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn
index 5ea813726d8aa1bf375b912e8a388075206060b6..475b66da305d1c7e455faf90ce15ed45beed101d 100644
--- a/chrome/test/BUILD.gn
+++ b/chrome/test/BUILD.gn
@@ -554,6 +554,7 @@ if (!is_android) {
           "../browser/ui/views/tabs/tab_drag_controller_interactive_uitest.h",
           "../browser/ui/views/toolbar/toolbar_action_view_interactive_uitest.cc",
           "../browser/ui/views/toolbar/toolbar_button_interactive_uitest.cc",
+          "../browser/ui/views/toolbar/toolbar_view_browsertest.cc",
           "../browser/ui/views/toolbar/toolbar_view_interactive_uitest.cc",

[sky, 2016/10/11 19:24:09]

It is confusing to have both a toolbar_view_browsertest and
toolbar_view_interactive_uitest both in interactive ui tests. How about
copy/paste the code from toolbar_view_browsertest.cc into interactive_uitest and
nuke toolbar_view_browsertest.cc?

[Qiang(Joe) Xu, 2016/10/11 22:49:29]

Done.

           "../browser/ui/views/translate/translate_bubble_test_utils_views.cc",
           "base/interactive_test_utils_views.cc",
@@ -2117,7 +2118,6 @@ test("browser_tests") {
           "../browser/ui/views/passwords/password_dialog_view_browsertest.cc",
           "../browser/ui/views/task_manager_view_browsertest.cc",
           "../browser/ui/views/toolbar/browser_actions_container_browsertest.cc",
-          "../browser/ui/views/toolbar/toolbar_view_browsertest.cc",
           "../browser/ui/views/translate/translate_bubble_view_browsertest.cc",
           "../browser/ui/views/web_dialog_view_browsertest.cc",
         ]