[Android] Sets up a coverage system for java using EMMA

build/java_apk.gypi

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # This file is meant to be included into a target to provide a rule
 # to build Android APKs in a consistent manner.
 #
 # To use this, create a gyp target with the following form:
 # {
 #   'target_name': 'my_package_apk',
@@ skipping 78 matching lines @@
     'compile_stamp': '<(intermediate_dir)/compile.stamp',
     'jar_stamp': '<(intermediate_dir)/jar.stamp',
     'obfuscate_stamp': '<(intermediate_dir)/obfuscate.stamp',
     'strip_stamp': '<(intermediate_dir)/strip.stamp',
     'classes_dir': '<(intermediate_dir)/classes',
     'javac_includes': [],
     'jar_excluded_classes': [],
     'jar_path': '<(PRODUCT_DIR)/lib.java/<(jar_name)',
     'obfuscated_jar_path': '<(intermediate_dir)/obfuscated.jar',
     'dex_path': '<(intermediate_dir)/classes.dex',
+    'emma_device_jar': '<(android_sdk_root)/tools/lib/emma_device.jar',

[frankf, 2013/08/01 19:57:41]

This is defined in bunch of places. Is there a way to define it once?

[gkanwar1, 2013/08/07 19:24:56]

Unfortunately I don't think there's a simple way to share this constant between
GYP and the ant files. That said, I removed it from java.gypi, since we don't
need it in there any more. It's now referred to once in java_apk.gypi and once
in apk-package.xml. I think that's the best we can get.

     'android_manifest_path%': '<(java_in_dir)/AndroidManifest.xml',
     'push_stamp': '<(intermediate_dir)/push.stamp',
     'link_stamp': '<(intermediate_dir)/link.stamp',
     'package_resources_stamp': '<(intermediate_dir)/package_resources.stamp',
     'codegen_input_paths': [],
     'unsigned_apk_path': '<(intermediate_dir)/<(apk_name)-unsigned.apk',
     'final_apk_path%': '<(PRODUCT_DIR)/apks/<(apk_name).apk',
     'incomplete_apk_path': '<(intermediate_dir)/<(apk_name)-incomplete.apk',
     'source_dir': '<(java_in_dir)/src',
     'apk_install_record': '<(intermediate_dir)/apk_install.record.stamp',
@@ skipping 14 matching lines @@
         ['gyp_managed_install == 1', {
           'apk_package_native_libs_dir': '<(intermediate_dir)/libs.managed',
         }, {
           'apk_package_native_libs_dir': '<(intermediate_dir)/libs',
         }],
       ],
     },
     'native_lib_target%': '',
     'apk_package_native_libs_dir': '<(apk_package_native_libs_dir)',
     'unsigned_standalone_apk_path': '<(unsigned_standalone_apk_path)',
+    'conditions': [
+      ['emma_instrument != 0', {
+        'classes_final_dir': '<(intermediate_dir)/classes_instr',
+      }, {
+        'classes_final_dir': '<(intermediate_dir)/classes',
+      }]
+    ]
   },
   # Pass the jar path to the apk's "fake" jar target.  This would be better as
   # direct_dependent_settings, but a variable set by a direct_dependent_settings
   # cannot be lifted in a dependent to all_dependent_settings.
   'all_dependent_settings': {
     'variables': {
       'apk_output_jar_path': '<(PRODUCT_DIR)/lib.java/<(jar_name)',
     },
   },
   'conditions': [
@@ skipping 222 matching lines @@
           'action': [
             'python', '<(DEPTH)/build/android/gyp/apk_install.py',
             '--android-sdk-tools=<(android_sdk_tools)',
             '--apk-path=<(incomplete_apk_path)',
             '--build-device-configuration=<(build_device_config_path)',
             '--install-record=<(apk_install_record)',
           ],
         },
       ],
     }],
+    ['emma_instrument != 0', {
+      'actions': [
+        {
+          'action_name': 'emma_instr_<(_target_name)',
+          'message': 'Instrumenting <(_target_name) classes',
+          'inputs': [
+            '<(compile_stamp)',
+            '<(DEPTH)/build/android/ant/apk-instr.xml',
+            '<(DEPTH)/build/android/gyp/ant.py',
+            '<(DEPTH)/build/android/gyp/util/build_utils.py',
+          ],
+          'outputs': [
+            '<(intermediate_dir)/coverage.em',
+            '<(classes_final_dir)',
+          ],
+          'action': [
+            'python', '<(DEPTH)/build/android/gyp/ant.py',
+            '-DCLASSES_DIR=<(classes_dir)',
+            '-DCLASSES_FINAL_DIR=<(classes_final_dir)',
+            '-DEMMA_COVERAGE_FILE=<(intermediate_dir)/coverage.em',
+            '-DEMMA_SOURCES_FILE=<(intermediate_dir)/emma_sources.txt',
+            '-DJAVA_SRC=<(java_in_dir)/src',
+            '-DADDITIONAL_SRC=>(additional_src_dirs)',
+            '-DGENERATED_SRC=>(generated_src_dirs)',
+            '-DANDROID_SDK_ROOT=<(android_sdk_root)',
+            '-Dbasedir=.',
+            '-buildfile',
+            '<(DEPTH)/build/android/ant/apk-instr.xml',
+            'instr-classes',
+          ]
+        },
+      ],
+    }],
   ],
   'actions': [
     {
       'action_name': 'ant_codegen_<(_target_name)',
       'message': 'Generating R.java for <(_target_name)',
       'conditions': [
         ['is_test_apk == 1', {
           'variables': {
             'additional_res_dirs=': [],
             'additional_res_packages=': [],
@@ skipping 71 matching lines @@
         '--chromium-code=<(chromium_code)',
         '--stamp=<(compile_stamp)',
 
         # TODO(newt): remove this once http://crbug.com/177552 is fixed in ninja.
         '--ignore=>!(echo \'>(_inputs)\' | md5sum)',
       ],
     },
     {
       'action_name': 'jar_<(_target_name)',
       'message': 'Creating <(_target_name) jar',
+      'conditions': [
+        ['emma_instrument != 0', {
+          'inputs': [ '<(intermediate_dir)/coverage.em', ]
+        }, {
+          'inputs': [ '<(compile_stamp)' ]
+        }]
+      ],
       'inputs': [
         '<(DEPTH)/build/android/gyp/util/build_utils.py',
         '<(DEPTH)/build/android/gyp/util/md5_check.py',
         '<(DEPTH)/build/android/gyp/jar.py',
-        '<(compile_stamp)',
       ],
       'outputs': [
         '<(jar_stamp)',
       ],
       'action': [
         'python', '<(DEPTH)/build/android/gyp/jar.py',
-        '--classes-dir=<(classes_dir)',
+        '--classes-dir=<(classes_final_dir)',
         '--jar-path=<(jar_path)',
         '--excluded-classes=<(jar_excluded_classes)',
         '--stamp=<(jar_stamp)',
 
         # TODO(newt): remove this once http://crbug.com/177552 is fixed in ninja.
         '--ignore=>!(echo \'>(_inputs)\' | md5sum)',
       ]
     },
     {
       'action_name': 'ant_obfuscate_<(_target_name)',
@@ skipping 41 matching lines @@
       ],
     },
     {
       'action_name': 'dex_<(_target_name)',
       'variables': {
         'conditions': [
           ['proguard_enabled == "true"', {
             'input_paths': [ '<(obfuscate_stamp)' ],
             'proguard_enabled_input_path': '<(obfuscated_jar_path)',
           }],
+          ['emma_instrument != 0', {
+            'dex_input_paths': [ '<(emma_device_jar)' ],
+            'input_paths': [ '<(intermediate_dir)/coverage.em' ],
+          }, {
+            'input_paths': [ '<(compile_stamp)' ]
+          }],
         ],
-        'input_paths': [ '<(compile_stamp)' ],
         'dex_input_paths': [ '>@(library_dexed_jars_paths)' ],
-        'dex_generated_input_dirs': [ '<(classes_dir)' ],
+        'dex_generated_input_dirs': [ '<(classes_final_dir)' ],
         'output_path': '<(dex_path)',
       },
       'includes': [ 'android/dex_action.gypi' ],
     },
     {
       'action_name': 'ant package resources',
       'message': 'Packaging resources for <(_target_name) APK.',
       'inputs': [
         '<(DEPTH)/build/android/ant/apk-package-resources.xml',
         '<(DEPTH)/build/android/gyp/util/build_utils.py',
@@ skipping 62 matching lines @@
       'action': [
         'python', '<(DEPTH)/build/android/gyp/ant.py',
         '-quiet',
         '-DANDROID_SDK_ROOT=<(android_sdk_root)',
         '-DAPK_NAME=<(apk_name)',
         '-DCONFIGURATION_NAME=<(CONFIGURATION_NAME)',
         '-DNATIVE_LIBS_DIR=<(apk_package_native_libs_dir)',
         '-DOUT_DIR=<(intermediate_dir)',
         '-DSOURCE_DIR=<(source_dir)',
         '-DUNSIGNED_APK_PATH=<(unsigned_apk_path)',
+        '-DEMMA_INSTRUMENT=<(emma_instrument)',
 
         '-Dbasedir=.',
         '-buildfile',
         '<(DEPTH)/build/android/ant/apk-package.xml',
 
         # Add list of inputs to the command line, so if inputs change
         # (e.g. if a Java file is removed), the command will be re-run.
         # TODO(newt): remove this once crbug.com/177552 is fixed in ninja.
         '-DTHIS_IS_IGNORED=>!(echo \'>(_inputs)\' | md5sum)',
       ]
     },
   ],
 }