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

Unified Diff: tools/parse_llvm_coverage.py

Issue 1246043002: Coverage: add a trace for not_covered_lines per file (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Adjust format Created 5 years, 5 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/parse_llvm_coverage.py
diff --git a/tools/parse_llvm_coverage.py b/tools/parse_llvm_coverage.py
index 2eadddd3cb1f7615758df257d92b5955648462c2..f721bd50781f7bcc154d691ea0d6153c17a983a4 100755
--- a/tools/parse_llvm_coverage.py
+++ b/tools/parse_llvm_coverage.py
@@ -107,12 +107,14 @@ def _nanobench_json(results, properties, key):
_testname(f): {
'coverage': {
'percent': percent,
+ 'lines_not_covered': not_covered_lines,
'options': {
'fullname': f,
'dir': os.path.dirname(f),
+ 'source_type': 'coverage',
},
},
- } for percent, f in results
+ } for percent, not_covered_lines, f in results
}
return rv
@@ -150,6 +152,7 @@ def _get_per_file_summaries(line_by_line):
covered_lines += 1
if total_lines > 0:
per_file.append((float(covered_lines)/float(total_lines)*100.0,
+ total_lines - covered_lines,
filepath))
return per_file
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698