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

Side by Side Diff: build/secondary/testing/gtest/BUILD.gn

Issue 1758653004: [GN] Set testonly unconditionally. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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 unified diff | Download patch
« no previous file with comments | « build/secondary/testing/gmock/BUILD.gn ('k') | third_party/WebKit/Source/platform/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 config("gtest_config") { 5 config("gtest_config") {
6 visibility = [ 6 visibility = [
7 ":*", 7 ":*",
8 "//testing/gmock:*", # gmock also shares this config. 8 "//testing/gmock:*", # gmock also shares this config.
9 ] 9 ]
10 10
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 config("gtest_warnings") { 61 config("gtest_warnings") {
62 if (is_win && is_clang) { 62 if (is_win && is_clang) {
63 # The Mutex constructor initializer list in gtest-port.cc is incorrectly 63 # The Mutex constructor initializer list in gtest-port.cc is incorrectly
64 # ordered. See 64 # ordered. See
65 # https://groups.google.com/d/msg/googletestframework/S5uSV8L2TX8/U1FaTDa6J6 sJ. 65 # https://groups.google.com/d/msg/googletestframework/S5uSV8L2TX8/U1FaTDa6J6 sJ.
66 cflags = [ "-Wno-reorder" ] 66 cflags = [ "-Wno-reorder" ]
67 } 67 }
68 } 68 }
69 69
70 static_library("gtest") { 70 static_library("gtest") {
71 # TODO http://crbug.com/412064 enable this flag all the time. 71 testonly = true
72 testonly = !is_component_build
73 sources = [ 72 sources = [
74 "include/gtest/gtest-death-test.h", 73 "include/gtest/gtest-death-test.h",
75 "include/gtest/gtest-message.h", 74 "include/gtest/gtest-message.h",
76 "include/gtest/gtest-param-test.h", 75 "include/gtest/gtest-param-test.h",
77 "include/gtest/gtest-printers.h", 76 "include/gtest/gtest-printers.h",
78 "include/gtest/gtest-spi.h", 77 "include/gtest/gtest-spi.h",
79 "include/gtest/gtest-test-part.h", 78 "include/gtest/gtest-test-part.h",
80 "include/gtest/gtest-typed-test.h", 79 "include/gtest/gtest-typed-test.h",
81 "include/gtest/gtest.h", 80 "include/gtest/gtest.h",
82 "include/gtest/gtest_pred_impl.h", 81 "include/gtest/gtest_pred_impl.h",
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
132 configs -= [ "//build/config/compiler:chromium_code" ] 131 configs -= [ "//build/config/compiler:chromium_code" ]
133 configs += [ 132 configs += [
134 "//build/config/compiler:no_chromium_code", 133 "//build/config/compiler:no_chromium_code",
135 134
136 # Must be after no_chromium_code for warning flags to be ordered correctly. 135 # Must be after no_chromium_code for warning flags to be ordered correctly.
137 ":gtest_warnings", 136 ":gtest_warnings",
138 ] 137 ]
139 } 138 }
140 139
141 source_set("gtest_main") { 140 source_set("gtest_main") {
142 # TODO http://crbug.com/412064 enable this flag all the time. 141 testonly = true
143 testonly = !is_component_build
144 sources = [ 142 sources = [
145 "src/gtest_main.cc", 143 "src/gtest_main.cc",
146 ] 144 ]
147 deps = [ 145 deps = [
148 ":gtest", 146 ":gtest",
149 ] 147 ]
150 } 148 }
OLDNEW
« no previous file with comments | « build/secondary/testing/gmock/BUILD.gn ('k') | third_party/WebKit/Source/platform/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698