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

Unified Diff: build/config/android/internal_rules.gni

Issue 2605023003: [android] Disable NewApi lint warning when building with emma coverage enabled. (Closed)
Patch Set: Created 4 years 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
« no previous file with comments | « build/android/gyp/lint.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/android/internal_rules.gni
diff --git a/build/config/android/internal_rules.gni b/build/config/android/internal_rules.gni
index f6052ae4229546aeb26da53541a149f19594add3..b247dd6f1f146d869fec668ce1f4e0c8221b309f 100644
--- a/build/config/android/internal_rules.gni
+++ b/build/config/android/internal_rules.gni
@@ -689,9 +689,12 @@ if (enable_java_templates) {
rebase_path(_config_path, root_build_dir),
"--result-path",
rebase_path(_result_path, root_build_dir),
- "--enable",
]
+ if (defined(invoker.disable)) {
+ args += [ "--disable=${invoker.disable}" ]
+ }
+
if (defined(invoker.create_cache) && invoker.create_cache) {
args += [
"--create-cache",
@@ -2479,6 +2482,9 @@ if (enable_java_templates) {
java_sources_file = _java_sources_file
}
deps = _accumulated_deps
+ if (_emma_instrument) {
agrieve 2017/01/03 13:49:11 nit: can you add a comment pointing to the bug her
jbudorick 2017/01/03 14:04:56 Done.
+ disable = [ "NewApi" ]
+ }
}
if (_run_findbugs) {
« no previous file with comments | « build/android/gyp/lint.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698