Insert logcat as part of test result for android 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 f8d37e76c0223bc5b3f001268120ccd5cbca42e7..2a5125c0985e4e783fa779ae0b0c6bec417e0556 100644
--- a/build/android/pylib/instrumentation/instrumentation_test_instance.py
+++ b/build/android/pylib/instrumentation/instrumentation_test_instance.py
@@ -442,6 +442,9 @@ class InstrumentationTestInstance(test_instance.TestInstance):
     self._store_tombstones = False
     self._initializeTombstonesAttributes(args)
 
+    self._json_results_file = None
+    self._initializeJsonResultsFileAttributes(args)
+
   def _initializeApkAttributes(self, args, error_func):
     if args.apk_under_test:
       apk_under_test_path = args.apk_under_test
@@ -594,6 +597,9 @@ class InstrumentationTestInstance(test_instance.TestInstance):
   def _initializeTombstonesAttributes(self, args):
     self._store_tombstones = args.store_tombstones
 
+  def _initializeJsonResultsFileAttributes(self, args):
+    self._json_results_file = args.json_results_file

[jbudorick, 2016/11/23 16:25:21]

nit: I would store this as self._should_save_logcat or something, since we don't
care what the json_results_file is, just whether or not it exists for the
purposes of saving the logcation.

[BigBossZhiling, 2016/11/29 00:40:42]

Done.

+
   @property
   def additional_apks(self):
     return self._additional_apks
@@ -627,6 +633,10 @@ class InstrumentationTestInstance(test_instance.TestInstance):
     return self._flags
 
   @property
+  def json_results_file(self):
+    return self._json_results_file
+
+  @property
   def package_info(self):
     return self._package_info