Fixed coverage dir for instrumentation tests.

build/android/pylib/instrumentation/instrumentation_test_instance.py

Index: build/android/pylib/instrumentation/instrumentation_test_instance.py
diff --git a/build/android/pylib/instrumentation/instrumentation_test_instance.py b/build/android/pylib/instrumentation/instrumentation_test_instance.py
index 610f084ba054b787488f49cc00ef860143db3396..89e7b4b2997db7c9cc692c448d73ee8a4a282921 100644
--- a/build/android/pylib/instrumentation/instrumentation_test_instance.py
+++ b/build/android/pylib/instrumentation/instrumentation_test_instance.py
@@ -239,6 +239,9 @@ class InstrumentationTestInstance(test_instance.TestInstance):
     self._timeout_scale = None
     self._initializeTestControlAttributes(args)
 
+    self._coverage_directory = None
+    self._initializeTestCoverageAttributes(args)
+
   def _initializeApkAttributes(self, args, error_func):
     if args.apk_under_test:
       apk_under_test_path = args.apk_under_test
@@ -388,6 +391,12 @@ class InstrumentationTestInstance(test_instance.TestInstance):
   def _initializeTestControlAttributes(self, args):
     self._timeout_scale = args.timeout_scale or 1
 
+  def _initializeTestCoverageAttributes(self, args):
+    self._coverage_directory = args.coverage_dir
+
+  def CoverageDirectory(self):

[mikecase (-- gone --), 2016/05/11 17:39:58]

Nit: To keep with the style of much of the other code I would change this to

@property
def coverage_directory(self):
  return self._coverage_directory

[jbudorick, 2016/05/17 00:07:33]

this, and also alphabetize it within the other properties

[BigBossZhiling, 2016/05/20 18:38:08]

Done.

+    return self._coverage_directory
+
   @property
   def additional_apks(self):
     return self._additional_apks