Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2858)

Unified Diff: chrome/test/BUILD.gn

Issue 2210573002: browser_test: Add a browser_tests_runner target for running tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/chrome_tests.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/BUILD.gn
diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn
index 786686fb177dd68b238a03e05b0d681fb79aaf48..a522e68d091833dcd14f62386b3965c857fc988d 100644
--- a/chrome/test/BUILD.gn
+++ b/chrome/test/BUILD.gn
@@ -865,13 +865,42 @@ if (!is_android) {
}
}
-test("browser_tests") {
+static_library("browser_tests_runner") {
+ testonly = true
sources = [
"base/browser_tests_main.cc",
]
- sources += rebase_path(chrome_tests_gypi_values.chrome_browser_tests_sources,
- ".",
- "//chrome")
+
+ deps = [
+ ":test_support",
+ ]
+
+ if (use_aura) {
+ sources += [
+ "base/mash_browser_tests_main.cc",
+ "base/mash_browser_tests_main.h",
+ "base/mojo_test_connector.cc",
+ "base/mojo_test_connector.h",
+ ]
+
+ deps += [
+ "//services/shell/background:lib",
+ "//services/shell/background/tests:test_support",
+ ]
+
+ data_deps = [
+ "//chrome/app:mojo_manifests",
+ "//mash/session",
+ ]
+
+ defines = [ "MOJO_SHELL_CLIENT" ]
+ }
+}
+
+test("browser_tests") {
+ sources = rebase_path(chrome_tests_gypi_values.chrome_browser_tests_sources,
+ ".",
+ "//chrome")
configs += [
"//build/config:precompiled_headers",
@@ -879,6 +908,7 @@ test("browser_tests") {
]
deps = [
+ ":browser_tests_runner",
":test_support",
"//base",
"//components/sync:test_support_sync_api",
@@ -1478,25 +1508,6 @@ test("browser_tests") {
}
if (use_aura) {
- sources += [
- "base/mash_browser_tests_main.cc",
- "base/mash_browser_tests_main.h",
- "base/mojo_test_connector.cc",
- "base/mojo_test_connector.h",
- ]
-
- deps += [
- "//services/shell/background:lib",
- "//services/shell/background/tests:test_support",
- ]
-
- data_deps += [
- "//chrome/app:mojo_manifests",
- "//mash/session",
- ]
-
- defines += [ "MOJO_SHELL_CLIENT" ]
-
if (enable_wifi_display) {
sources += rebase_path(
chrome_tests_gypi_values.chrome_browser_tests_display_source_apitest,
@@ -1536,6 +1547,7 @@ if (!is_android) {
defines = [ "HAS_OUT_OF_PROC_TEST_RUNNER" ]
deps = [
+ ":browser_tests_runner",
":sync_integration_test_support",
":test_support",
":test_support_ui",
« no previous file with comments | « chrome/chrome_tests.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698