[Android] Detect when remote service loses contact with device.

build/android/pylib/remote/device/remote_device_test_run.py

 # Copyright 2014 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.
 
 """Run specific test on specific environment."""
 
 import json
 import logging
 import os
+import re
 import shutil
 import string
 import tempfile
 import time
 import zipfile
 
 from pylib.base import test_run
 from pylib.remote.device import appurify_constants
 from pylib.remote.device import appurify_sanitized
 from pylib.remote.device import remote_device_helper
@@ skipping 309 matching lines @@
 
     Raises:
       KeyError: If appurify_results/logcat.txt file cannot be found in
                 downloaded zip.
     """
     zip_file = self._DownloadTestResults(None)
     with zipfile.ZipFile(zip_file) as z:
       try:
         logcat = z.read('appurify_results/logcat.txt')
         printable_logcat = ''.join(c for c in logcat if c in string.printable)
+        logging.log(level, '---Logging remote device logcat---')

[jbudorick, 2015/08/06 16:31:36]

+1

[rnephew (Reviews Here), 2015/08/06 16:42:07]

Done.

         for line in printable_logcat.splitlines():
           logging.log(level, line)
+        logging.log(level, '---Ending remote device logcat---')
       except KeyError:
         logging.error('No logcat found.')
+
+  def _DidDeviceGoOffline(self):
+    device_offline = re.compile('error: device not found')

[jbudorick, 2015/08/06 16:31:36]

This should be compiled into a constant at module scope.

[rnephew (Reviews Here), 2015/08/06 16:42:07]

Done.

+    zip_file = self._DownloadTestResults(None)
+    with zipfile.ZipFile(zip_file) as z:
+      adb_trace_log = z.read('adb_trace.log')
+      if any(device_offline.search(l) for l in adb_trace_log.splitlines()):
+        logging.critical('---Logging remote device adb trace log---')

[jbudorick, 2015/08/06 16:31:36]

The function name doesn't imply anything about logging, and it's inconsistent
with how we handle the logcat. I would prefer doing this in a separate function.

[rnephew (Reviews Here), 2015/08/06 16:42:07]

Done.

+        for line in adb_trace_log.splitlines():
+          logging.critical(line)
+        logging.critical('---Ending remote device adb trace log---')
+        return True
+    return False