Add lots of testonly annotations to the GN build.

BUG=
R=hclam@chromium.org

Review URL: https://codereview.chromium.org/547793002

Cr-Commit-Position: refs/heads/master@{#293774}

ash/BUILD.gn

@@ -165,6 +165,7 @@ component("ash_with_content") {
 }
 
 static_library("test_support") {
+  testonly = true
   sources = gypi_values.ash_test_support_sources
 
   deps = [
@@ -198,6 +199,7 @@ static_library("test_support") {
 }
 
 static_library("ash_shell_lib") {
+  testonly = true
   sources = gypi_values.ash_shell_lib_sources
 
   deps = [
@@ -338,6 +340,7 @@ test("ash_unittests") {
 }
 
 executable("ash_shell") {
+  testonly = true
   sources = [
     "shell/shell_main.cc",
   ]

base/BUILD.gn

@@ -1014,6 +1014,7 @@ source_set("prefs") {
 }
 
 source_set("prefs_test_support") {
+  testonly = true
   sources = [
     "prefs/mock_pref_change_callback.cc",
     "prefs/mock_pref_change_callback.h",

base/test/BUILD.gn

@@ -9,6 +9,7 @@ if (is_android) {
 }
 
 source_set("test_support") {
+  testonly = true
   sources = [
     "expectations/expectation.cc",
     "expectations/expectation.h",
@@ -134,6 +135,7 @@ config("perf_test_config") {
 }
 
 source_set("test_support_perf") {
+  testonly = true
   sources = [
     "run_all_perftests.cc",
   ]
@@ -147,6 +149,7 @@ source_set("test_support_perf") {
 }
 
 source_set("run_all_unittests") {
+  testonly = true
   sources = [
     "run_all_unittests.cc",
   ]

cc/BUILD.gn

@@ -495,6 +495,7 @@ component("cc") {
 }
 
 source_set("test_support") {
+  testonly = true
   sources = [
     "test/animation_test_common.cc",
     "test/animation_test_common.h",

components/data_reduction_proxy/browser/BUILD.gn

@@ -35,6 +35,7 @@ static_library("browser") {
 }
 
 static_library("test_support") {
+  testonly = true
   sources = [
     "data_reduction_proxy_params_test_utils.cc",
     "data_reduction_proxy_params_test_utils.h",
@@ -54,6 +55,7 @@ static_library("test_support") {
 }
 
 source_set("unit_tests") {
+  testonly = true
   sources = [
     "data_reduction_proxy_auth_request_handler_unittest.cc",
     "data_reduction_proxy_config_service_unittest.cc",

components/proximity_auth/BUILD.gn

@@ -15,6 +15,7 @@ static_library("proximity_auth") {
 }
 
 source_set("unit_tests") {
+  testonly = true
   sources = [
     "proximity_auth_system_unittest.cc",
   ]

content/shell/BUILD.gn

@@ -20,6 +20,7 @@ declare_args() {
 }
 
 static_library("content_shell_lib") {
+  testonly = true
   sources = [
     "android/shell_jni_registrar.cc",
     "android/shell_jni_registrar.h",
@@ -428,6 +429,7 @@ repack("pak") {
 }
 
 executable("content_shell") {
+  testonly = true
   # TODO(GYP) mac resource bundle stuff for this target.
   # TODO(GYP) Windows content shell settings:
   #  - Manifest.

content/test/BUILD.gn

@@ -14,6 +14,7 @@ content_tests_gypi_values = exec_script(
 
 # GYP version //content/content_tests.gypi:test_support_content
 static_library("test_support") {
+  testonly = true
   deps = [
     "//content/public/app:both",
     "//content/public/browser",
@@ -139,6 +140,7 @@ if (!is_ios) {
 
   # GYP version //content/content_tests.gypi:layouttest_support_content
   static_library("layouttest_support") {
+    testonly = true
     sources = rebase_path(content_tests_gypi_values.layouttest_support_content_sources,
                           ".", "//content")
 

dbus/BUILD.gn

@@ -55,6 +55,7 @@ proto_library("test_proto") {
 # This target contains mocks that can be used to write unit tests without
 # issuing actual D-Bus calls.
 source_set("test_support") {
+  testonly = true
   sources = [
     "mock_bus.cc",
     "mock_bus.h",
@@ -111,6 +112,7 @@ test("dbus_unittests") {
 }
 
 executable("dbus_test_server") {
+  testonly = true
   sources = [
     "test_server.cc",
     "test_service.cc",

extensions/BUILD.gn

@@ -55,6 +55,7 @@ grit("extensions_renderer_resources") {
 }
 
 source_set("test_support") {
+  testonly = true
   sources = [
     "browser/api/dns/mock_host_resolver_creator.cc",
     "browser/api/dns/mock_host_resolver_creator.h",

google_apis/BUILD.gn

@@ -169,6 +169,7 @@ source_set("google_apis") {
 }
 
 source_set("test_support") {
+  testonly = true
   sources = [
     "gaia/fake_gaia.cc",
     "gaia/fake_gaia.h",

google_apis/gcm/BUILD.gn

@@ -67,6 +67,7 @@ proto_library("proto") {
 }
 
 static_library("test_support") {
+  testonly = true
   sources = [
     "base/fake_encryptor.cc",
     "base/fake_encryptor.h",
@@ -89,6 +90,7 @@ static_library("test_support") {
 
 # A standalone MCS (mobile connection server) client.
 executable("mcs_probe") {
+  testonly = true
   sources = [
     "tools/mcs_probe.cc",
   ]

jingle/BUILD.gn

@@ -108,6 +108,7 @@ if (enable_webrtc || !is_android) {
   # GYP version: jingle/jingle.gyp:jingle_unittests
   # TODO(GYP): Convert to executable when its dependencies are linkable.
   source_set("jingle_unittests") {
+    testonly = true
     sources = [
       "glue/channel_socket_adapter_unittest.cc",
       "glue/chrome_async_socket_unittest.cc",

media/cast/BUILD.gn

@@ -169,6 +169,7 @@ source_set("receiver") {
 }
 
 source_set("test_support") {
+  testonly = true
   sources = [
 # TODO(hclam): FFmpeg.
 #    "test/fake_media_source.cc",
@@ -283,6 +284,7 @@ test("cast_unittests") {
 }
 
 executable("generate_barcode_video") {
+  testonly = true
   sources = [
     "test/utility/generate_barcode_video.cc",
   ]
@@ -295,6 +297,7 @@ executable("generate_barcode_video") {
 }
 
 executable("generate_timecode_audio") {
+  testonly = true
   sources = [
     "test/utility/generate_timecode_audio.cc",
   ]
@@ -307,6 +310,7 @@ executable("generate_timecode_audio") {
 }
 
 executable("udp_proxy") {
+  testonly = true
   sources = [
     "test/utility/udp_proxy_main.cc",
   ]

mojo/common/test/BUILD.gn

@@ -4,6 +4,7 @@
 
 # GYP version: mojo/mojo_base.gyp:mojo_run_all_unittests
 source_set("run_all_unittests") {
+  testonly = true
   deps = [
     ":test_support_impl",
     "//base",
@@ -18,6 +19,7 @@ source_set("run_all_unittests") {
 
 # GYP version: mojo/mojo_base.gyp:mojo_test_support_impl
 source_set("test_support_impl") {
+  testonly = true
   deps = [
     "//base",
   ]
@@ -30,6 +32,7 @@ source_set("test_support_impl") {
 
 # GYP version: mojo/mojo_base.gyp:mojo_common_test_support
 source_set("test_support") {
+  testonly = true
   deps = [
     "//base",
     "//base/test:test_support",

mojo/embedder/BUILD.gn

@@ -53,6 +53,7 @@ source_set("embedder") {
 
 source_set("embedder_unittests") {
   visibility = [ "//mojo/system:mojo_system_unittests" ]
+  testonly = true
 
   deps = [
     "//base",

mojo/services/public/cpp/view_manager/lib/BUILD.gn

@@ -6,6 +6,7 @@ import("//build/config/ui.gni")
 
 # GYP version: mojo/mojo_services.gypi:mojo_view_manager_run_unittests
 source_set("run_unittests") {
+  testonly = true
   sources = [
     "view_manager_test_suite.cc",
     "view_manager_test_suite.h",

net/BUILD.gn

@@ -594,6 +594,7 @@ static_library("http_server") {
 }
 
 executable("dump_cache") {
+  testonly = true
   sources = [
     "tools/dump_cache/cache_dumper.cc",
     "tools/dump_cache/cache_dumper.h",
@@ -618,6 +619,7 @@ executable("dump_cache") {
 }
 
 source_set("test_support") {
+  testonly = true
   sources = [
     "base/capturing_net_log.cc",
     "base/capturing_net_log.h",
@@ -798,6 +800,7 @@ if (use_v8_in_net) {
 
 if (!is_ios && !is_android) {
   executable("crash_cache") {
+    testonly = true
     sources = [ "tools/crash_cache/crash_cache.cc" ]
     configs += [ ":net_win_size_truncation" ]
     deps = [
@@ -808,6 +811,7 @@ if (!is_ios && !is_android) {
   }
 
   executable("crl_set_dump") {
+    testonly = true
     sources = [ "tools/crl_set_dump/crl_set_dump.cc" ]
     configs += [ ":net_win_size_truncation" ]
     deps = [
@@ -817,6 +821,7 @@ if (!is_ios && !is_android) {
   }
 
   executable("dns_fuzz_stub") {
+    testonly = true
     sources = [ "tools/dns_fuzz_stub/dns_fuzz_stub.cc" ]
     configs += [ ":net_win_size_truncation" ]
     deps = [
@@ -826,6 +831,7 @@ if (!is_ios && !is_android) {
   }
 
   executable("gdig") {
+    testonly = true
     sources = [
       "tools/gdig/file_net_log.cc",
       "tools/gdig/gdig.cc",
@@ -837,6 +843,7 @@ if (!is_ios && !is_android) {
   }
 
   executable("get_server_time") {
+    testonly = true
     sources = [ "tools/get_server_time/get_server_time.cc" ]
     configs += [ ":net_win_size_truncation" ]
     deps = [
@@ -849,6 +856,7 @@ if (!is_ios && !is_android) {
 
   if (use_v8_in_net) {
     executable("net_watcher") {
+      testonly = true
       sources = [ "tools/net_watcher/net_watcher.cc" ]
       deps = [
         ":net",
@@ -867,6 +875,7 @@ if (!is_ios && !is_android) {
   }
 
   executable("run_testserver") {
+    testonly = true
     sources = [ "tools/testserver/run_testserver.cc" ]
     deps = [
       ":net",  # TODO(brettw) bug 363749: this shouldn't be necessary. It's not
@@ -879,6 +888,7 @@ if (!is_ios && !is_android) {
   }
 
   executable("stress_cache") {
+    testonly = true
     sources = [ "disk_cache/blockfile/stress_cache.cc" ]
     configs += [ ":net_win_size_truncation" ]
     deps = [
@@ -912,6 +922,7 @@ if (is_linux) {
   }
 
   static_library("flip_in_mem_edsm_server_base") {
+    testonly = true
     sources = [
       "tools/dump_cache/url_to_filename_encoder.cc",
       "tools/dump_cache/url_to_filename_encoder.h",
@@ -955,6 +966,7 @@ if (is_linux) {
   }
 
   executable("flip_in_mem_edsm_server_unittests") {
+    testonly = true
     sources = [
       "tools/flip_server/flip_test_utils.cc",
       "tools/flip_server/flip_test_utils.h",
@@ -974,6 +986,7 @@ if (is_linux) {
   }
 
   executable("flip_in_mem_edsm_server") {
+    testonly = true
     sources = [ "tools/flip_server/flip_in_mem_edsm_server.cc" ]
     deps = [
       ":flip_in_mem_edsm_server_base",
@@ -1062,6 +1075,7 @@ if (is_android) {
 
 if (is_android || is_linux) {
   executable("disk_cache_memory_test") {
+    testonly = true
     sources = [ "tools/disk_cache_memory_test/disk_cache_memory_test.cc" ]
     deps = [
       ":net",

ppapi/BUILD.gn

@@ -786,6 +786,7 @@ component("ppapi_host") {
 }
 
 static_library("ppapi_unittest_shared") {
+  testonly = true
   sources = [
     "proxy/ppapi_proxy_test.cc",
     "proxy/ppapi_proxy_test.h",

rlz/BUILD.gn

@@ -76,6 +76,7 @@ source_set("rlz_lib") {
 }
 
 source_set("test_support") {
+  testonly = true
   sources = [
     "test/rlz_test_helpers.cc",
     "test/rlz_test_helpers.h",

sync/BUILD.gn

@@ -709,6 +709,7 @@ static_library("test_support_sync_fake_server") {
 
 # GYP version: sync/sync_tests.gypi:test_support_sync_testserver
 static_library("test_support_sync_testserver") {
+  testonly = true
   sources = [
     "test/local_sync_test_server.cc",
     "test/local_sync_test_server.h",
@@ -730,6 +731,7 @@ static_library("test_support_sync_testserver") {
 
 # GYP version: sync/sync_tests.gypi:test_support_accounts_client
 static_library("test_support_accounts_client") {
+  testonly = true
   sources = [
     "test/accounts_client/test_accounts_client.cc",
     "test/accounts_client/test_accounts_client.h",
@@ -759,6 +761,7 @@ test("sync_endtoend_tests") {
 if (!is_ios) {
   # GYP version: sync/sync_tests.gypi:run_sync_testserver
   executable("run_sync_testserver") {
+    testonly = true
     sources = [ "tools/testserver/run_sync_testserver.cc" ]
 
     deps = [

ui/aura/BUILD.gn

@@ -142,6 +142,7 @@ component("aura") {
 }
 
 source_set("test_support") {
+  testonly = true
   sources = [
     "test/aura_test_base.cc",
     "test/aura_test_base.h",
@@ -209,6 +210,8 @@ source_set("test_support") {
 
 executable("demo") {
   output_name = "aura_demo"
+  testonly = true
+
   sources = [
     "demo/demo_main.cc",
   ]
@@ -238,6 +241,8 @@ executable("demo") {
 
 executable("bench") {
   output_name = "aura_bench"
+  testonly = true
+
   sources = [
     "bench/bench_main.cc",
   ]

ui/compositor/BUILD.gn

@@ -78,6 +78,7 @@ component("compositor") {
 }
 
 source_set("test_support") {
+  testonly = true
   sources = [
     "test/context_factories_for_test.cc",
     "test/context_factories_for_test.h",

ui/gfx/BUILD.gn

@@ -364,6 +364,7 @@ component("gfx") {
 # Looking for gfx_geometry? It's //ui/gfx/geometry:geometry
 
 source_set("test_support") {
+  testonly = true
   sources = [
     "test/fontconfig_util_linux.cc",
     "test/fontconfig_util_linux.h",

ui/message_center/BUILD.gn

@@ -147,6 +147,7 @@ component("message_center") {
 }
 
 static_library("test_support") {
+  testonly = true
   sources = [
     "fake_message_center.h",
     "fake_message_center.cc",

ui/views/BUILD.gn

@@ -104,6 +104,7 @@ component("views") {
 }
 
 static_library("test_support") {
+  testonly = true
   sources = gypi_values.views_test_support_sources
 
   deps = [

ui/views/controls/webview/BUILD.gn

@@ -40,6 +40,7 @@ component("webview") {
 }
 
 source_set("test_support") {
+  testonly = true
   sources = [
     "../../test/webview_test_helper.cc",
     "../../test/webview_test_helper.h",

ui/wm/BUILD.gn

@@ -93,6 +93,7 @@ component("wm") {
 }
 
 static_library("test_support") {
+  testonly = true
   sources = [
     "test/wm_test_helper.cc",
     "test/wm_test_helper.h",