| Index: ash/BUILD.gn
|
| diff --git a/ash/BUILD.gn b/ash/BUILD.gn
|
| index f142f8fb645cb1e22064b7d8a62ea3b928e3a4ea..d6642dd24088ed5dc7127f72d2c63d5b7d126917 100644
|
| --- a/ash/BUILD.gn
|
| +++ b/ash/BUILD.gn
|
| @@ -184,14 +184,9 @@
|
| ]
|
| }
|
|
|
| -# Internal target consumed by |test_support_with_content| and
|
| -# |test_support_without_content|. This target contains all the test support
|
| -# files, with the exception of an implementation of AshTestEnvironment.
|
| -# Consumers of ash should use one of |test_support_with_content| or
|
| -# |test_support_without_content|.
|
| -static_library("test_support_common") {
|
| +# TODO(msw): Move support code with content deps to test_support_with_content.
|
| +static_library("test_support") {
|
| testonly = true
|
| - visibility = [ ":*" ]
|
| sources = gypi_values.ash_test_support_sources
|
| configs += [ "//build/config:precompiled_headers" ]
|
|
|
| @@ -199,11 +194,14 @@
|
| ":ash",
|
| ]
|
| deps = [
|
| + ":test_support_with_content", # TODO(beng): reverse this direction.
|
| "//ash/resources",
|
| "//base:i18n",
|
| "//base/test:test_support",
|
| "//components/signin/core/account_id",
|
| "//components/user_manager:user_manager",
|
| + "//content/public/browser",
|
| + "//content/test:test_support",
|
| "//device/bluetooth",
|
| "//skia",
|
| "//testing/gtest",
|
| @@ -243,35 +241,13 @@
|
| }
|
| }
|
|
|
| -static_library("test_support_without_content") {
|
| - testonly = true
|
| - sources = [
|
| - "test/ash_test_environment_default.cc",
|
| - ]
|
| -
|
| - public_deps = [
|
| - ":test_support_common",
|
| - ]
|
| -
|
| - deps = [
|
| - "//base",
|
| - "//skia",
|
| - ]
|
| -}
|
| -
|
| static_library("test_support_with_content") {
|
| testonly = true
|
| sources = gypi_values.ash_test_support_with_content_sources
|
| - sources += [
|
| - "test/ash_test_environment_content.cc",
|
| - "test/ash_test_environment_content.h",
|
| - ]
|
| configs += [ "//build/config:precompiled_headers" ]
|
|
|
| - public_deps = [
|
| + deps = [
|
| ":ash_with_content",
|
| - ":test_support_common",
|
| - "//content/public/browser",
|
| "//content/test:test_support",
|
| "//skia",
|
| ]
|
| @@ -282,7 +258,7 @@
|
| configs += [ "//build/config:precompiled_headers" ]
|
| public_deps = [
|
| ":ash",
|
| - ":test_support_without_content",
|
| + ":test_support",
|
| ]
|
| sources = [
|
| "test/ash_interactive_ui_test_base.cc",
|
| @@ -290,7 +266,7 @@
|
| ]
|
| deps = [
|
| ":ash",
|
| - ":test_support_with_content",
|
| + ":test_support",
|
| "//base",
|
| "//skia",
|
| "//testing/gtest",
|
| @@ -307,7 +283,7 @@
|
| deps = [
|
| ":ash",
|
| ":ash_with_content",
|
| - ":test_support_without_content",
|
| + ":test_support",
|
| "//ash/common/strings",
|
| "//ash/resources",
|
| "//base",
|
| @@ -366,76 +342,15 @@
|
| }
|
| }
|
|
|
| -# This target is intended for tests that use content. In general very few tests
|
| -# need or use content and should be placed in |ash_unittests| instead.
|
| -test("ash_content_unittests") {
|
| - sources = [
|
| - "content/display/screen_orientation_controller_chromeos_unittest.cc",
|
| - "content/keyboard_overlay/keyboard_overlay_delegate_unittest.cc",
|
| - "content/keyboard_overlay/keyboard_overlay_view_unittest.cc",
|
| - "test/ash_unittests.cc",
|
| - ]
|
| - configs += [
|
| - "//build/config:precompiled_headers",
|
| - "//build/config/compiler:no_size_t_to_int_warning",
|
| - ]
|
| +test("ash_unittests") {
|
| + sources = gypi_values.ash_unittests_sources
|
| + configs += [ "//build/config:precompiled_headers" ]
|
|
|
| deps = [
|
| ":ash",
|
| ":ash_with_content",
|
| + ":test_support",
|
| ":test_support_with_content",
|
| - "//base",
|
| - "//base/test:test_support",
|
| - "//components/signin/core/account_id",
|
| - "//components/user_manager",
|
| - "//content/public/browser",
|
| - "//content/test:test_support",
|
| - "//mojo/edk/system",
|
| - "//skia",
|
| - "//testing/gtest",
|
| - "//ui/aura",
|
| - "//ui/aura:test_support",
|
| - "//ui/message_center",
|
| - "//ui/message_center:test_support",
|
| - "//ui/views",
|
| - "//ui/views:test_support",
|
| - "//ui/views/controls/webview:test_support",
|
| - "//ui/web_dialogs:test_support",
|
| - ]
|
| -
|
| - data_deps = [
|
| - "//ash/common/strings:ash_test_strings",
|
| - "//ash/resources:ash_test_resources_100_percent",
|
| - "//ash/resources:ash_test_resources_200_percent",
|
| - ]
|
| -
|
| - if (is_chromeos) {
|
| - deps += [
|
| - "//chromeos",
|
| - "//chromeos:power_manager_proto",
|
| - "//chromeos:test_support_without_gmock",
|
| - "//components/quirks",
|
| - "//device/bluetooth",
|
| - "//net:net",
|
| - "//ui/chromeos",
|
| - "//ui/display",
|
| - "//ui/display:test_support",
|
| - "//ui/display:test_util",
|
| - ]
|
| - }
|
| -}
|
| -
|
| -test("ash_unittests") {
|
| - sources = gypi_values.ash_unittests_sources
|
| - sources += [ "test/ash_test_environment_default.cc" ]
|
| - configs += [
|
| - "//build/config:precompiled_headers",
|
| - "//build/config/compiler:no_size_t_to_int_warning",
|
| - ]
|
| -
|
| - deps = [
|
| - ":ash",
|
| - ":test_support_without_content",
|
| "//ash/autoclick/common:autoclick",
|
| "//ash/common/strings",
|
| "//ash/resources",
|
| @@ -444,9 +359,10 @@
|
| "//base/test:test_support",
|
| "//components/signin/core/account_id",
|
| "//components/user_manager",
|
| + "//content/public/browser",
|
| + "//content/test:test_support",
|
| "//mojo/edk/system",
|
| "//skia",
|
| - "//testing/gmock",
|
| "//testing/gtest",
|
| "//third_party/icu",
|
| "//ui/accessibility",
|
| @@ -466,12 +382,15 @@
|
| "//ui/gfx",
|
| "//ui/gfx/geometry",
|
| "//ui/keyboard",
|
| + "//ui/keyboard:keyboard_with_content",
|
| "//ui/message_center",
|
| "//ui/message_center:test_support",
|
| "//ui/resources",
|
| "//ui/strings",
|
| "//ui/views",
|
| "//ui/views:test_support",
|
| + "//ui/views/controls/webview:test_support",
|
| + "//ui/web_dialogs:test_support",
|
| "//ui/wm",
|
| "//ui/wm:test_support",
|
| "//url",
|
| @@ -483,10 +402,36 @@
|
| "//ash/resources:ash_test_resources_200_percent",
|
| ]
|
|
|
| + if (!is_chromeos) {
|
| + sources -= [
|
| + # TODO(zork): fix this test to build on Windows. See: crosbug.com/26906
|
| + "focus_cycler_unittest.cc",
|
| +
|
| + # All tests for multiple displays: not supported on Windows Ash.
|
| + "wm/drag_window_resizer_unittest.cc",
|
| +
|
| + # Talks to ChromeOS system services.
|
| + "wm/lock_state_controller_unittest.cc",
|
| +
|
| + # Maximize mode is only available on Chrome OS.
|
| + "wm/maximize_mode/maximize_mode_controller_unittest.cc",
|
| + "wm/maximize_mode/maximize_mode_window_manager_unittest.cc",
|
| +
|
| + # Can't resize on Windows Ash. http://crbug.com/165962
|
| + "autoclick/autoclick_unittest.cc",
|
| + "magnifier/magnification_controller_unittest.cc",
|
| + "magnifier/partial_magnification_controller_unittest.cc",
|
| + "sticky_keys/sticky_keys_overlay_unittest.cc",
|
| + "virtual_keyboard_controller_unittest.cc",
|
| + "wm/workspace/workspace_window_resizer_unittest.cc",
|
| + ]
|
| + }
|
| +
|
| + configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
|
| +
|
| if (!use_x11) {
|
| sources -= [ "host/ash_window_tree_host_x11_unittest.cc" ]
|
| }
|
| -
|
| if (is_chromeos) {
|
| sources += [ "first_run/first_run_helper_unittest.cc" ]
|
| deps += [
|
| @@ -505,35 +450,17 @@
|
| sources -= [
|
| "accelerators/magnifier_key_scroller_unittest.cc",
|
| "accelerators/spoken_feedback_toggler_unittest.cc",
|
| -
|
| - # Can't resize on Windows Ash. http://crbug.com/165962
|
| - "autoclick/autoclick_unittest.cc",
|
| "display/resolution_notification_controller_unittest.cc",
|
| -
|
| - # TODO(zork): fix this test to build on Windows. See: crosbug.com/26906
|
| - "focus_cycler_unittest.cc",
|
| - "magnifier/magnification_controller_unittest.cc",
|
| - "magnifier/partial_magnification_controller_unittest.cc",
|
| - "sticky_keys/sticky_keys_overlay_unittest.cc",
|
| - "sticky_keys/sticky_keys_unittest.cc",
|
| "touch/touchscreen_util_unittest.cc",
|
| - "virtual_keyboard_controller_unittest.cc",
|
| -
|
| - # All tests for multiple displays: not supported on Windows Ash.
|
| - "wm/drag_window_resizer_unittest.cc",
|
| -
|
| - # Talks to ChromeOS system services.
|
| - "wm/lock_state_controller_unittest.cc",
|
| -
|
| - # Maximize mode is only available on Chrome OS.
|
| - "wm/maximize_mode/maximize_mode_controller_unittest.cc",
|
| - "wm/maximize_mode/maximize_mode_window_manager_unittest.cc",
|
| - "wm/workspace/workspace_window_resizer_unittest.cc",
|
| ]
|
| }
|
|
|
| if (!use_x11 || !is_chromeos) {
|
| sources -= [ "touch/touch_transformer_controller_unittest.cc" ]
|
| + }
|
| +
|
| + if (!is_chromeos) {
|
| + sources -= [ "sticky_keys/sticky_keys_unittest.cc" ]
|
| }
|
| }
|
|
|
|
|