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

chrome/test/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/chrome_build.gni")
 import("//build/config/compiler/compiler.gni")
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//chrome/common/features.gni")
@@ skipping 571 matching lines @@
           }
         }
       }
       if (is_chromeos || !use_x11) {
         sources -= [
           "//ui/views/widget/desktop_aura/desktop_window_tree_host_x11_interactive_uitest.cc",
           "//ui/views/widget/desktop_aura/x11_topmost_window_finder_interactive_uitest.cc",
         ]
       }
       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.
         sources += [ "//chrome/browser/renderer_host/site_per_process_text_input_browsertest.cc" ]
         deps += [ "//ui/touch_selection" ]
       }
     }
 
     if (!is_desktop_linux) {
       # Everything but desktop Linux.
@@ skipping 1513 matching lines @@
           "../browser/ui/views/extensions/bookmark_override_browsertest.cc",
           "../browser/ui/views/extensions/extension_install_dialog_view_browsertest.cc",
           "../browser/ui/views/extensions/extension_message_bubble_view_browsertest.cc",
           "../browser/ui/views/frame/browser_non_client_frame_view_browsertest.cc",
           "../browser/ui/views/frame/browser_view_browsertest.cc",
           "../browser/ui/views/location_bar/zoom_bubble_view_browsertest.cc",
           "../browser/ui/views/media_router/media_router_ui_browsertest.cc",
           "../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",
         ]
       }
     }
 
     if (is_linux && !is_component_build) {
       # Set rpath to find the CDM adapter even in a non-component build.
       configs += [ "//build/config/gcc:rpath_for_built_shared_libraries" ]
     }
@@ skipping 2890 matching lines @@
       "touch_hud",
       "ui",
     ]
 
     if (is_linux && !is_android) {
       deps += [ "//components/font_service:manifest" ]
       packaged_services += [ "font_service" ]
     }
   }
 }