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 0db7c5355dd922fb1c09466d1136b6c1bf0a3718..63efb8836a2b2fdf9cf401c34ffbcd6a31f1c3fa 100644
--- a/chrome/test/BUILD.gn
+++ b/chrome/test/BUILD.gn
@@ -589,6 +589,11 @@ if (!is_android) {
         ]
       }
       if (use_aura || is_mac) {
+        if (!is_mac) {

[sky, 2016/10/21 03:14:18]

I would rather you add an use_aura case. Having a conditional  like:

if (foo || bar) {
  if (!bar) {}
}

Is just confusing.

[Qiang(Joe) Xu, 2016/10/21 16:23:06]

Done.

+          sources +=
+              [ "//ui/views/widget/native_widget_aura_interactive_uitest.cc" ]
+        }
+
         # TODO(ekaramad): These tests are activated for aura and mac only. They
         # should be moved to 'interactive_ui_tests' sources for all platforms
         # once https://crbug.com/578168 is fixed.
@@ -2122,7 +2127,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",
         ]