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

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

Issue 1913593002: Enable emma code coverage for JUnit testcases. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixes Created 4 years, 8 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: build/config/android/internal_rules.gni
diff --git a/build/config/android/internal_rules.gni b/build/config/android/internal_rules.gni
index a11f4a04239c353b7a930115bf92955a19bbcfc7..2021d21e7b4cacd24d4ede50594820aa055ff1f2 100644
--- a/build/config/android/internal_rules.gni
+++ b/build/config/android/internal_rules.gni
@@ -240,6 +240,23 @@ template("java_binary_script") {
"--main-class",
_main_class,
]
+ if (emma_coverage) {
+ args += [
+ "--classpath",
+ rebase_path(
+ "../../third_party/android_tools_internal/sdk/tools/lib/emma.jar",
agrieve 2016/04/26 14:51:10 Use // syntax here: //third_party/android_tools_in
BigBossZhiling 2016/04/26 17:35:58 Done.
+ root_out_dir),
+ ]
+ args += [
+ "--noverify",
+ "true",
+ ]
+ } else {
+ args += [
+ "--noverify",
+ "false",
+ ]
+ }
if (defined(invoker.wrapper_script_args)) {
args += [ "--" ] + invoker.wrapper_script_args
}

Powered by Google App Engine
This is Rietveld 408576698