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

Side by Side Diff: webrtc/tools/BUILD.gn

Issue 2278723004: Add missing "//build/config/sanitizers:deps" to executable targets. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 3 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 | « webrtc/modules/audio_coding/BUILD.gn ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved. 1 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
2 # 2 #
3 # Use of this source code is governed by a BSD-style license 3 # Use of this source code is governed by a BSD-style license
4 # that can be found in the LICENSE file in the root of the source 4 # that can be found in the LICENSE file in the root of the source
5 # tree. An additional intellectual property rights grant can be found 5 # tree. An additional intellectual property rights grant can be found
6 # in the file PATENTS. All contributing project authors may 6 # in the file PATENTS. All contributing project authors may
7 # be found in the AUTHORS file in the root of the source tree. 7 # be found in the AUTHORS file in the root of the source tree.
8 8
9 import("//testing/test.gni") 9 import("//testing/test.gni")
10 import("//third_party/protobuf/proto_library.gni") 10 import("//third_party/protobuf/proto_library.gni")
(...skipping 246 matching lines...) Expand 10 before | Expand all | Expand 10 after
257 # Suppress warnings from the Chromium Clang plugin. 257 # Suppress warnings from the Chromium Clang plugin.
258 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details. 258 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
259 configs -= [ "//build/config/clang:find_bad_constructs" ] 259 configs -= [ "//build/config/clang:find_bad_constructs" ]
260 } 260 }
261 261
262 deps = [ 262 deps = [
263 "../system_wrappers:system_wrappers_default", 263 "../system_wrappers:system_wrappers_default",
264 "../test:channel_transport", 264 "../test:channel_transport",
265 "../test:test_support", 265 "../test:test_support",
266 "../voice_engine", 266 "../voice_engine",
267 "//build/config/sanitizers:deps",
267 "//build/win:default_exe_manifest", 268 "//build/win:default_exe_manifest",
268 "//testing/gtest", 269 "//testing/gtest",
269 "//third_party/gflags", 270 "//third_party/gflags",
270 ] 271 ]
271 } 272 }
272 273
273 executable("activity_metric") { 274 executable("activity_metric") {
274 testonly = true 275 testonly = true
275 sources = [ 276 sources = [
276 "agc/activity_metric.cc", 277 "agc/activity_metric.cc",
277 ] 278 ]
278 279
279 configs += [ "..:common_config" ] 280 configs += [ "..:common_config" ]
280 public_configs = [ "..:common_inherited_config" ] 281 public_configs = [ "..:common_inherited_config" ]
281 282
282 if (is_clang) { 283 if (is_clang) {
283 # Suppress warnings from the Chromium Clang plugin. 284 # Suppress warnings from the Chromium Clang plugin.
284 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details. 285 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
285 configs -= [ "//build/config/clang:find_bad_constructs" ] 286 configs -= [ "//build/config/clang:find_bad_constructs" ]
286 } 287 }
287 288
288 deps = [ 289 deps = [
289 "../modules/audio_processing", 290 "../modules/audio_processing",
290 "../system_wrappers:metrics_default", 291 "../system_wrappers:metrics_default",
292 "//build/config/sanitizers:deps",
291 "//build/win:default_exe_manifest", 293 "//build/win:default_exe_manifest",
292 "//testing/gtest", 294 "//testing/gtest",
293 "//third_party/gflags", 295 "//third_party/gflags",
294 ] 296 ]
295 } 297 }
296 298
297 executable("audio_e2e_harness") { 299 executable("audio_e2e_harness") {
298 testonly = true 300 testonly = true
299 sources = [ 301 sources = [
300 "e2e_quality/audio/audio_e2e_harness.cc", 302 "e2e_quality/audio/audio_e2e_harness.cc",
301 ] 303 ]
302 304
303 configs += [ "..:common_config" ] 305 configs += [ "..:common_config" ]
304 public_configs = [ "..:common_inherited_config" ] 306 public_configs = [ "..:common_inherited_config" ]
305 307
306 if (is_clang) { 308 if (is_clang) {
307 # Suppress warnings from the Chromium Clang plugin. 309 # Suppress warnings from the Chromium Clang plugin.
308 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details. 310 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
309 configs -= [ "//build/config/clang:find_bad_constructs" ] 311 configs -= [ "//build/config/clang:find_bad_constructs" ]
310 } 312 }
311 313
312 deps = [ 314 deps = [
313 "../system_wrappers:system_wrappers_default", 315 "../system_wrappers:system_wrappers_default",
314 "../test:channel_transport", 316 "../test:channel_transport",
315 "../voice_engine", 317 "../voice_engine",
318 "//build/config/sanitizers:deps",
316 "//build/win:default_exe_manifest", 319 "//build/win:default_exe_manifest",
317 "//testing/gtest", 320 "//testing/gtest",
318 "//third_party/gflags", 321 "//third_party/gflags",
319 ] 322 ]
320 } 323 }
321 324
322 test("tools_unittests") { 325 test("tools_unittests") {
323 testonly = true 326 testonly = true
324 327
325 sources = [ 328 sources = [
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
369 ] 372 ]
370 outputs = [ 373 outputs = [
371 "$root_build_dir/{{source_file_part}}", 374 "$root_build_dir/{{source_file_part}}",
372 ] 375 ]
373 deps = [ 376 deps = [
374 "..:rtc_event_log_proto", 377 "..:rtc_event_log_proto",
375 ] 378 ]
376 } 379 }
377 } 380 }
378 } 381 }
OLDNEW
« no previous file with comments | « webrtc/modules/audio_coding/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698