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

Unified Diff: scripts/slave/recipe_modules/chromium/chromium_memory_fyi.py

Issue 612753004: Create MSan config for Chromium recipe module + add WebRTC MSan recipe. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/chromium/chromium_memory_fyi.py
diff --git a/scripts/slave/recipe_modules/chromium/chromium_memory_fyi.py b/scripts/slave/recipe_modules/chromium/chromium_memory_fyi.py
index d9cbb0bd044e25c4de84ea8579b5877a2ec13a0d..68e503ffef0ec0e137c23d8e90cf2e8eb0266628 100644
--- a/scripts/slave/recipe_modules/chromium/chromium_memory_fyi.py
+++ b/scripts/slave/recipe_modules/chromium/chromium_memory_fyi.py
@@ -10,13 +10,7 @@ SPEC = {
},
'builders': {
'Chromium Linux MSan Builder': {
- 'recipe_config': 'chromium_clang',
- 'GYP_DEFINES': {
- 'msan': 1,
- 'msan_track_origins': 0,
- 'use_instrumented_libraries': 1,
- 'instrumented_libraries_jobs': 10,
- },
+ 'recipe_config': 'chromium_msan',
'chromium_config_kwargs': {
'BUILD_CONFIG': 'Release',
'TARGET_BITS': 64,
@@ -27,12 +21,7 @@ SPEC = {
},
},
'Linux MSan Tests': {
- 'recipe_config': 'chromium_clang',
- 'GYP_DEFINES': {
- # Required on testers to pass the right runtime flags.
- # TODO(earthdok): make this part of a chromium_msan recipe config.
- 'msan': 1,
- },
+ 'recipe_config': 'chromium_msan',
'chromium_config_kwargs': {
'BUILD_CONFIG': 'Release',
'TARGET_BITS': 64,
@@ -47,12 +36,7 @@ SPEC = {
},
},
'Linux MSan Browser (1)': {
- 'recipe_config': 'chromium_clang',
- 'GYP_DEFINES': {
- # Required on testers to pass the right runtime flags.
- # TODO(earthdok): make this part of a chromium_msan recipe config.
- 'msan': 1,
- },
+ 'recipe_config': 'chromium_msan',
'chromium_config_kwargs': {
'BUILD_CONFIG': 'Release',
'TARGET_BITS': 64,
@@ -67,12 +51,7 @@ SPEC = {
},
},
'Linux MSan Browser (2)': {
- 'recipe_config': 'chromium_clang',
- 'GYP_DEFINES': {
- # Required on testers to pass the right runtime flags.
- # TODO(earthdok): make this part of a chromium_msan recipe config.
- 'msan': 1,
- },
+ 'recipe_config': 'chromium_msan',
'chromium_config_kwargs': {
'BUILD_CONFIG': 'Release',
'TARGET_BITS': 64,
@@ -87,12 +66,7 @@ SPEC = {
},
},
'Linux MSan Browser (3)': {
- 'recipe_config': 'chromium_clang',
- 'GYP_DEFINES': {
- # Required on testers to pass the right runtime flags.
- # TODO(earthdok): make this part of a chromium_msan recipe config.
- 'msan': 1,
- },
+ 'recipe_config': 'chromium_msan',
'chromium_config_kwargs': {
'BUILD_CONFIG': 'Release',
'TARGET_BITS': 64,
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/chromium/config.py » ('j') | scripts/slave/recipe_modules/chromium/config.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698