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

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

Issue 1544803003: GN: Set --coverage-dir in bin/run_* script when emma_coverage = true (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add gypi support Created 5 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/test_runner.gypi ('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 4673bf4567ba405962d4694c0024b5bd8580b76f..3416243004889a263ab0a8db01006fc1797705fb 100644
--- a/build/config/android/internal_rules.gni
+++ b/build/config/android/internal_rules.gni
@@ -1991,6 +1991,14 @@ template("test_runner_script") {
"--apk-under-test",
"@FileArg($_rebased_build_config:deps_info:tested_apk_path)",
]
+ if (emma_coverage) {
+ # Set a default coverage output directory (can be overridden by user
+ # passing the same flag).
+ test_runner_args += [
+ "--coverage-dir",
+ rebase_path("$root_out_dir/coverage", root_build_dir),
+ ]
+ }
} else {
assert(false, "Invalid test type: $_test_type.")
}
« no previous file with comments | « build/android/test_runner.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698