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

build/java.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 Java in a consistent manner.
 #
 # To use this, create a gyp target with the following form:
 # {
 #   'target_name': 'my-package_java',
@@ skipping 37 matching lines @@
 {
   'dependencies': [
     '<(DEPTH)/build/android/setup.gyp:build_output_dirs'
   ],
   'variables': {
     'android_jar': '<(android_sdk)/android.jar',
     'input_jars_paths': [ '<(android_jar)' ],
     'additional_src_dirs': [],
     'javac_includes': [],
     'jar_name': '<(_target_name).jar',
-    'jar_path': '<(PRODUCT_DIR)/lib.java/<(jar_name)',
+    'jar_dir': '<(PRODUCT_DIR)/lib.java',
+    'jar_path': '<(intermediate_dir)/<(jar_name)',
+    'jar_final_path': '<(jar_dir)/<(jar_name)',
     'jar_excluded_classes': [ '*/R.class', '*/R##*.class' ],
     'additional_input_paths': [],
     'dex_path': '<(PRODUCT_DIR)/lib.java/<(_target_name).dex.jar',
     'generated_src_dirs': ['>@(generated_R_dirs)'],
     'generated_R_dirs': [],
     'has_java_resources%': 0,
     'java_strings_grd%': '',
     'res_extra_dirs': [],
     'res_extra_files': [],
     'res_v14_verify_only%': 0,
     'resource_input_paths': ['>@(res_extra_files)'],
     'intermediate_dir': '<(SHARED_INTERMEDIATE_DIR)/<(_target_name)',
     'classes_dir': '<(intermediate_dir)/classes',
     'compile_stamp': '<(intermediate_dir)/compile.stamp',
   },
   # This all_dependent_settings is used for java targets only. This will add the
   # jar path to the classpath of dependent java targets.
   'all_dependent_settings': {
     'variables': {
-      'input_jars_paths': ['<(jar_path)'],
+      'input_jars_paths': ['<(jar_final_path)'],
       'library_dexed_jars_paths': ['<(dex_path)'],
     },
   },
   'conditions': [
     ['has_java_resources == 1', {
       'variables': {
         'res_dir': '<(java_in_dir)/res',
         'res_crunched_dir': '<(intermediate_dir)/res_crunched',
         'res_v14_compatibility_stamp': '<(intermediate_dir)/res_v14_compatibility.stamp',
         'res_v14_compatibility_dir': '<(intermediate_dir)/res_v14_compatibility',
@@ skipping 180 matching lines @@
         'python', '<(DEPTH)/build/android/gyp/jar.py',
         '--classes-dir=<(classes_dir)',
         '--jar-path=<(jar_path)',
         '--excluded-classes=<(jar_excluded_classes)',
 
         # TODO(newt): remove this once http://crbug.com/177552 is fixed in ninja.
         '--ignore=>!(echo \'>(_inputs)\' | md5sum)',
       ]
     },
     {
+      'action_name': 'instr_jar_<(_target_name)',

[cjhopman, 2013/08/10 00:11:47]

Extract this action to its own file in build/android (like
build/android/dex_action.gypi). Then expose those variables that should be
configured by the caller (mostly the things that are different between the use
here and the one in java.gypi). 

[gkanwar1, 2013/08/12 18:19:10]

Done.

+      'message': 'Instrumenting <(_target_name) jar',
+      'variables': {
+        'conditions': [
+          ['emma_instrument != 0', {
+            'instr_action': 'instrument_jar',
+            'extra_instr_args': [
+              '--intermediate-dir=<(intermediate_dir)',
+              '--coverage-file=<(intermediate_dir)/coverage.em',
+              '--sources-file=<(intermediate_dir)/emma_sources.txt',
+              '--sources=<(java_in_dir)/src >(additional_src_dirs) >(generated_src_dirs)',
+              '--src-root=<(DEPTH)',
+              '--sdk-root=<(android_sdk_root)',
+            ],
+          }, {
+            'instr_action': 'copy',
+            'extra_instr_args': [],
+          }]
+        ]
+      },
+      'inputs': [
+        '<(jar_path)',
+        '<(DEPTH)/build/android/gyp/instr.py',
+        '<(DEPTH)/build/android/pylib/utils/command_option_parser.py',
+      ],
+      'outputs': [
+        '<(jar_final_path)',
+      ],
+      'action': [
+        'python', '<(DEPTH)/build/android/gyp/instr.py',
+        '<(instr_action)',
+        '--input-path=<(jar_path)',
+        '--output-path=<(jar_dir)',
+        '<@(extra_instr_args)',
+      ]
+    },
+    {
       'action_name': 'jar_toc_<(_target_name)',
       'message': 'Creating <(_target_name) jar.TOC',
       'inputs': [
         '<(DEPTH)/build/android/gyp/util/build_utils.py',
         '<(DEPTH)/build/android/gyp/util/md5_check.py',
         '<(DEPTH)/build/android/gyp/jar_toc.py',
-        '<(jar_path)',
+        '<(jar_final_path)',
       ],
       'outputs': [
-        '<(jar_path).TOC',
+        '<(jar_final_path).TOC',
       ],
       'action': [
         'python', '<(DEPTH)/build/android/gyp/jar_toc.py',
-        '--jar-path=<(jar_path)',
-        '--toc-path=<(jar_path).TOC',
+        '--jar-path=<(jar_final_path)',
+        '--toc-path=<(jar_final_path).TOC',
 
         # TODO(newt): remove this once http://crbug.com/177552 is fixed in ninja.
         '--ignore=>!(echo \'>(_inputs)\' | md5sum)',
       ]
     },
     {
       'action_name': 'dex_<(_target_name)',
       'variables': {
-        'dex_input_paths': [ '<(jar_path)' ],
+        'conditions': [
+          ['emma_instrument != 0', {
+            'dex_no_locals': 1,
+          }],
+        ],
+        'dex_input_paths': [ '<(jar_final_path)' ],
         'output_path': '<(dex_path)',
       },
       'includes': [ 'android/dex_action.gypi' ],
     },
   ],
 }