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

Unified Diff: masters/master.client.webrtc.fyi/master_linux_cfg.py

Issue 218493009: WebRTC+libyuv: switch to use_allocator GYP variable. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: masters/master.client.webrtc.fyi/master_linux_cfg.py
diff --git a/masters/master.client.webrtc.fyi/master_linux_cfg.py b/masters/master.client.webrtc.fyi/master_linux_cfg.py
index 197ab19249666c214ad667791ea87ee6076a253e..f74bc38566e2fe0ac46e848303c51678fffbd14d 100644
--- a/masters/master.client.webrtc.fyi/master_linux_cfg.py
+++ b/masters/master.client.webrtc.fyi/master_linux_cfg.py
@@ -50,7 +50,7 @@ F('linux_tsan2_factory', linux().WebRTCFactory(
'tsan_suppressions_file':
'src/tools/valgrind-webrtc/tsan_v2/suppressions.txt',
'gclient_env': {
- 'GYP_DEFINES': ('tsan=1 linux_use_tcmalloc=0 '
+ 'GYP_DEFINES': ('tsan=1 use_allocator=none '
'release_extra_cflags="-gline-tables-only"'),
phoglund_chromium 2014/04/01 07:08:06 This is different than the others, which are just
kjellander_chromium 2014/04/01 08:16:08 I think it's actually better for us to use -g here
}}))
@@ -74,7 +74,7 @@ F('linux_lsan_factory', linux().WebRTCFactory(
'gclient_env': {
'GYP_DEFINES': ('asan=1 '
'lsan=1 '
- 'linux_use_tcmalloc=0 '
+ 'use_allocator=none '
'release_extra_cflags="-gline-tables-only" ')}}))
phoglund_chromium 2014/04/01 07:08:06 Same here
kjellander_chromium 2014/04/01 08:16:08 Yepp, changed this too.
« no previous file with comments | « masters/master.client.libyuv/master_builders_cfg.py ('k') | masters/master.client.webrtc/master_linux_cfg.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698