GN: Add "//build/config/sanitizers:deps" to executable targets

webrtc/modules/audio_coding/BUILD.gn

 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
 #
 # Use of this source code is governed by a BSD-style license
 # that can be found in the LICENSE file in the root of the source
 # tree. An additional intellectual property rights grant can be found
 # in the file PATENTS.  All contributing project authors may
 # be found in the AUTHORS file in the root of the source tree.
 
 import("../../build/webrtc.gni")
 import("audio_coding.gni")
@@ skipping 1113 matching lines @@
       deps += [ "//testing/android/native_test:native_test_native_code" ]
       shard_timeout = 900
     }
 
     deps += [
       ":isac_fix",
       ":webrtc_opus",
       "../../system_wrappers:system_wrappers_default",
       "../../test:test_support_main",
       "../audio_processing/",
+      "//build/config/sanitizers:deps",

[kjellander_webrtc, 2016/08/17 12:19:22]

Shouldn't be needed for targets using the test template. Sure it's needed?

[ehmaldonado_webrtc, 2016/08/17 12:37:05]

You're right, sorry.

       "//testing/gtest",
     ]
   }
 
   source_set("neteq_test_support") {
     testonly = true
     sources = [
       "neteq/tools/neteq_external_decoder_test.cc",
       "neteq/tools/neteq_external_decoder_test.h",
       "neteq/tools/neteq_performance_test.cc",
@@ skipping 140 matching lines @@
       # TODO(hlundin): Make RTPencode use ACM to encode files.
       ":cng",
       ":g711",
       ":g722",
       ":ilbc",
       ":isac",
       ":neteq_test_tools",
       ":pcm16b",
       ":webrtc_opus",
       "../../common_audio",
+      "//build/config/sanitizers:deps",
     ]
 
     configs += [ ":RTPencode_config" ]
 
     sources = [
       "neteq/test/RTPencode.cc",
     ]
 
     include_dirs = [
       "neteq/include",
@@ skipping 10 matching lines @@
 
   executable("RTPchange") {
     testonly = true
 
     sources = [
       "neteq/test/RTPchange.cc",
     ]
 
     deps = [
       ":neteq_test_tools",
+      "//build/config/sanitizers:deps",
     ]
   }
 
   executable("rtpcat") {
     testonly = true
 
     sources = [
       "neteq/tools/rtpcat.cc",
     ]
 
     deps = [
       "../../system_wrappers:system_wrappers_default",
       "../../test:rtp_test_utils",
+      "//build/config/sanitizers:deps",
       "//testing/gtest",
     ]
   }
 
   executable("RTPtimeshift") {
     testonly = true
 
     sources = [
       "neteq/test/RTPtimeshift.cc",
     ]
 
     deps = [
       ":neteq_test_tools",
+      "//build/config/sanitizers:deps",
       "//testing/gtest",
     ]
   }
 
   executable("RTPjitter") {
     testonly = true
     deps = [
       "../..:webrtc_common",
+      "//build/config/sanitizers:deps",
       "//testing/gtest",
     ]
     sources = [
       "neteq/test/RTPjitter.cc",
     ]
   }
 
   executable("rtp_analyze") {
     testonly = true
 
     sources = [
       "neteq/tools/rtp_analyze.cc",
     ]
 
     deps = [
       ":neteq",
       ":neteq_unittest_tools",
       ":pcm16b",
       "../../system_wrappers:system_wrappers_default",
+      "//build/config/sanitizers:deps",
       "//testing/gtest",
       "//third_party/gflags:gflags",
     ]
 
     if (is_clang) {
       # Suppress warnings from Chrome's Clang plugins.
       # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
       configs -= [ "//build/config/clang:find_bad_constructs" ]
     }
   }
 
   executable("neteq_opus_quality_test") {
     testonly = true
 
     sources = [
       "neteq/test/neteq_opus_quality_test.cc",
     ]
 
     deps = [
       ":neteq",
       ":neteq_test_support",
       ":neteq_unittest_tools",
       ":webrtc_opus",
       "../../test:test_support_main",

[kjellander_webrtc, 2016/08/17 12:19:22]

Missing for this target (and others in this file).

[ehmaldonado_webrtc, 2016/08/17 12:37:05]

Ok, I'll do, but I'm not sure if it's really needed.

       "//testing/gtest",
       "//third_party/gflags",
     ]
   }
 
   executable("neteq_speed_test") {
     testonly = true
 
     sources = [
       "neteq/test/neteq_speed_test.cc",
     ]
 
     deps = [
       ":neteq",
       ":neteq_test_support",
       "../../system_wrappers:system_wrappers_default",
       "../../test:test_support",
+      "//build/config/sanitizers:deps",
       "//third_party/gflags",
     ]
   }
 
   executable("audio_classifier_test") {
     testonly = true
     sources = [
       "neteq/test/audio_classifier_test.cc",
     ]
     deps = [
       ":neteq",
       ":webrtc_opus",
       "../../system_wrappers:system_wrappers_default",
+      "//build/config/sanitizers:deps",
     ]
   }
 
   executable("neteq_ilbc_quality_test") {
     testonly = true
 
     sources = [
       "neteq/test/neteq_ilbc_quality_test.cc",
     ]
 
@@ skipping 46 matching lines @@
   executable("isac_fix_test") {
     testonly = true
 
     sources = [
       "codecs/isac/fix/test/kenny.cc",
     ]
 
     deps = [
       ":isac_fix",
       "../../test:test_support",
+      "//build/config/sanitizers:deps",
     ]
 
     if (is_win) {
       cflags = [
         # Disable warnings to enable Win64 build, issue 1323.
         "/wd4267",  # size_t to int truncation
       ]
     }
   }
 }