diff --git a/ninjalog_uploader.py b/ninjalog_uploader.py index 1f1d98dc0..3aa0239e6 100755 --- a/ninjalog_uploader.py +++ b/ninjalog_uploader.py @@ -30,12 +30,13 @@ from third_party.six.moves import http_client from third_party.six.moves.urllib import error from third_party.six.moves.urllib import request -# These build configs affect build performance a lot. -# TODO(https://crbug.com/900161): Add 'blink_symbol_level' and -# 'enable_js_type_check'. -WHITELISTED_CONFIGS = ('symbol_level', 'use_goma', 'is_debug', +# These build configs affect build performance. +ALLOWLISTED_CONFIGS = ('symbol_level', 'use_goma', 'is_debug', 'is_component_build', 'enable_nacl', 'host_os', - 'host_cpu', 'target_os', 'target_cpu') + 'host_cpu', 'target_os', 'target_cpu', + 'blink_symbol_level', 'is_java_debug', + 'treat_warnings_as_errors', 'disable_android_lint', + 'use_errorprone_java_compiler', 'incremental_install') def IsGoogler(server): @@ -54,7 +55,7 @@ def ParseGNArgs(gn_args): for config in configs: key = config["name"] - if key not in WHITELISTED_CONFIGS: + if key not in ALLOWLISTED_CONFIGS: continue if 'current' in config: build_configs[key] = config['current']['value'] diff --git a/ninjalog_uploader_wrapper.py b/ninjalog_uploader_wrapper.py index e55fca49a..fbfca3f29 100755 --- a/ninjalog_uploader_wrapper.py +++ b/ninjalog_uploader_wrapper.py @@ -43,7 +43,7 @@ def SaveConfig(config): def ShowMessage(countdown): whitelisted = '\n'.join( - [' * %s' % config for config in ninjalog_uploader.WHITELISTED_CONFIGS]) + [' * %s' % config for config in ninjalog_uploader.ALLOWLISTED_CONFIGS]) print(""" Your ninjalog will be uploaded to build stats server. The uploaded log will be used to analyze user side build performance.