[Android] Adds python option for EMMA coverage

build/android/pylib/constants.py

Index: build/android/pylib/constants.py
diff --git a/build/android/pylib/constants.py b/build/android/pylib/constants.py
index bf35fff6e7b7c9f9b454f2972301d244ddcbf2ff..7d9335034bf55e89dc0134a79b2ccf44b6da6a4d 100644
--- a/build/android/pylib/constants.py
+++ b/build/android/pylib/constants.py
@@ -14,7 +14,6 @@ DIR_SOURCE_ROOT = os.path.abspath(os.path.join(os.path.dirname(__file__),
 ISOLATE_DEPS_DIR = os.path.join(DIR_SOURCE_ROOT, 'isolate_deps_dir')
 EMULATOR_SDK_ROOT = os.path.abspath(os.path.join(DIR_SOURCE_ROOT, os.pardir,
                                                  os.pardir))
-
 CHROME_PACKAGE = 'com.google.android.apps.chrome'
 CHROME_ACTIVITY = 'com.google.android.apps.chrome.Main'
 CHROME_DEVTOOLS_SOCKET = 'chrome_devtools_remote'
@@ -94,6 +93,12 @@ def SetBuildType(build_type):
   os.environ['BUILDTYPE'] = build_type
 
 
+def GetCoverageDir(build_type=None):

[frankf, 2013/08/20 18:30:47]

Where is this used?

[gkanwar1, 2013/08/20 18:37:13]

This is used by the buildbot scripts. I put it with this CL, since that one will
be the most time-constrained at the end, and the fewer changes included in it,
the better.

[frankf, 2013/08/20 18:47:43]

Why do you need to take build_type as parameter?

On 2013/08/20 18:37:13, gkanwar1 wrote:

[gkanwar1, 2013/08/20 18:56:18]

Test-generated files (like logs, etc.) generally are placed into the
out/<target> folder, rather than the top-level out folder, so this needs to
accept build_type to place it in the correct folder.

+  if not build_type:
+    build_type = GetBuildType()
+  return os.path.join(DIR_SOURCE_ROOT, 'out', build_type, 'coverage')
+
+
 def _GetADBPath():
   if os.environ.get('ANDROID_SDK_ROOT'):
     return 'adb'