Make lsan and tsan suppressions files overridable.

build/config/sanitizers/sanitizers.gni

Index: build/config/sanitizers/sanitizers.gni
diff --git a/build/config/sanitizers/sanitizers.gni b/build/config/sanitizers/sanitizers.gni
index 40ffb94d09885f6b6ed056860db472e0bc186bfe..a2334f145a5063e4c5cab5361b36a9609b1b0edf 100644
--- a/build/config/sanitizers/sanitizers.gni
+++ b/build/config/sanitizers/sanitizers.gni
@@ -2,7 +2,16 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+# See https://bugs.chromium.org/p/webrtc/issues/detail?id=6236.

[kjellander_chromium, 2016/08/22 14:59:21]

Remove our bug refererence.

+import("//build_overrides/build.gni")
+
 declare_args() {
+  #

[kjellander_chromium, 2016/08/22 14:59:21]

Please add comments.

+  lsan_suppressions_file = lsan_suppressions_file_override

[kjellander_chromium, 2016/08/22 14:59:21]

I found this pattern confusing, is it really needed? I understand Edward took it
from https://cs.chromium.org/chromium/src/build/config/mac/mac_sdk.gni though.

[Dirk Pranke, 2016/08/22 17:21:50]

You would only need this if you  need to set the value differently per-repo and
*also* want to be able to change the value in an args.gn file.

Most overrides only need one or the other. Given that this isn't a declare_arg()
value in chromium, it seems unlikely that it needs to be a declare_arg() in
webrtc as well?

[kjellander_chromium, 2016/08/22 21:00:51]

Ah, now I remember that regarding the Mac SDK version.

Edward: just remove the changes in this file and include
import("//build_overrides/build.gni") straight from
	build/config/sanitizers/BUILD.gn then.

+
+  #
+  tsan_suppressions_file = tsan_suppressions_file_override
+
   # Compile for Address Sanitizer to find memory bugs.
   is_asan = false