Changed behavior when no files require coverage.

build/android/emma_coverage_stats.py

Index: build/android/emma_coverage_stats.py
diff --git a/build/android/emma_coverage_stats.py b/build/android/emma_coverage_stats.py
index 8500890703498679c76431b0ced124cd2101bc84..c113c72aa675acbcc7486e881da1b107b005a3c4 100755
--- a/build/android/emma_coverage_stats.py
+++ b/build/android/emma_coverage_stats.py
@@ -450,14 +450,12 @@ def GenerateCoverageReport(line_coverage_file, out_file_path, coverage_dir):
   files_for_coverage = {f: lines
                         for f, lines in potential_files_for_coverage.iteritems()
                         if _EmmaCoverageStats.NeedsCoverage(f)}
-  if not files_for_coverage:
-    logging.info('No Java files requiring coverage were included in %s.',
-                 line_coverage_file)
-    return
-
-  code_coverage = _EmmaCoverageStats(coverage_dir, files_for_coverage.keys())
-  coverage_results = code_coverage.GetCoverageDict(
-      files_for_coverage)
+
+  coverage_results = {}
+  if files_for_coverage:
+    code_coverage = _EmmaCoverageStats(coverage_dir, files_for_coverage.keys())
+    coverage_results = code_coverage.GetCoverageDict(
+        files_for_coverage)
 

[mikecase (-- gone --), 2015/08/11 23:43:55]

nit: Add back the logging statement you removed in an else block here.

[estevenson1, 2015/08/11 23:48:16]

Done.

   with open(out_file_path, 'w+') as out_status_file:
     json.dump(coverage_results, out_status_file)