gn: Add missing testonly for chromium and fuchsia styles

Bug: crashpad:79, crashpad:196
Change-Id: I8354b8430cfe4728a635991fb59fcc8ef8652773
Reviewed-on: https://chromium-review.googlesource.com/835468
Reviewed-by: Mark Mentovai <mark@chromium.org>
Commit-Queue: Scott Graham <scottmg@chromium.org>
This commit is contained in:
Scott Graham 2017-12-19 16:48:09 -08:00 committed by Commit Bot
parent 41c7ace7e8
commit e7ebdf019e

View File

@ -17,23 +17,29 @@ import("../../build/test.gni")
if (crashpad_is_in_chromium) { if (crashpad_is_in_chromium) {
group("gtest") { group("gtest") {
testonly = true
public_deps = [ public_deps = [
"//testing/gtest", "//testing/gtest",
] ]
} }
group("gmock") { group("gmock") {
testonly = true
public_deps = [ public_deps = [
"//testing/gmock", "//testing/gmock",
] ]
} }
} else if (crashpad_is_in_fuchsia) { } else if (crashpad_is_in_fuchsia) {
# TODO(scottmg): Fuchsia doesn't have a third_party/gmock, and has a
# pre-gmock-integration gtest.
group("gtest") { group("gtest") {
testonly = true
public_deps = [ public_deps = [
"//third_party/gtest", "//third_party/gtest",
] ]
} }
group("gmock") {
testonly = true
# TODO(scottmg): Fuchsia doesn't have a third_party/gmock, and has a
# pre-gmock-integration gtest.
}
} else if (crashpad_is_standalone) { } else if (crashpad_is_standalone) {
config("gtest_private_config") { config("gtest_private_config") {
visibility = [ ":*" ] visibility = [ ":*" ]
@ -84,7 +90,9 @@ if (crashpad_is_in_chromium) {
] ]
sources -= [ "gtest/googletest/src/gtest-all.cc" ] sources -= [ "gtest/googletest/src/gtest-all.cc" ]
public_configs = [ ":gtest_public_config" ] public_configs = [ ":gtest_public_config" ]
configs -= [ "//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors" ] configs -= [
"//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors",
]
configs += [ ":gtest_private_config" ] configs += [ ":gtest_private_config" ]
} }
@ -121,7 +129,9 @@ if (crashpad_is_in_chromium) {
"gtest/googletest/test/production.cc", "gtest/googletest/test/production.cc",
"gtest/googletest/test/production.h", "gtest/googletest/test/production.h",
] ]
configs -= [ "//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors" ] configs -= [
"//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors",
]
configs += [ ":gtest_private_config" ] configs += [ ":gtest_private_config" ]
deps = [ deps = [
":gtest", ":gtest",
@ -163,7 +173,9 @@ if (crashpad_is_in_chromium) {
"gtest/googletest/test/gtest-param-test_test.cc", "gtest/googletest/test/gtest-param-test_test.cc",
"gtest/googletest/test/gtest-param-test_test.h", "gtest/googletest/test/gtest-param-test_test.h",
] ]
configs -= [ "//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors" ] configs -= [
"//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors",
]
configs += [ ":gtest_private_config" ] configs += [ ":gtest_private_config" ]
deps = [ deps = [
":gtest", ":gtest",
@ -291,7 +303,9 @@ if (crashpad_is_in_chromium) {
] ]
sources -= [ "gtest/googlemock/src/gmock-all.cc" ] sources -= [ "gtest/googlemock/src/gmock-all.cc" ]
public_configs = [ ":gmock_public_config" ] public_configs = [ ":gmock_public_config" ]
configs -= [ "//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors" ] configs -= [
"//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors",
]
configs += [ ":gmock_private_config" ] configs += [ ":gmock_private_config" ]
deps = [ deps = [
":gtest", ":gtest",
@ -364,7 +378,9 @@ if (crashpad_is_in_chromium) {
sources = [ sources = [
"gtest/googlemock/test/gmock_stress_test.cc", "gtest/googlemock/test/gmock_stress_test.cc",
] ]
configs -= [ "//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors" ] configs -= [
"//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors",
]
configs += [ ":gmock_private_config" ] configs += [ ":gmock_private_config" ]
deps = [ deps = [
":gmock", ":gmock",