Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(120)

Unified Diff: telemetry/telemetry/internal/platform/android_platform_backend.py

Issue 2162963002: [polymer] Merge of master into polymer10-migration (Closed) Base URL: git@github.com:catapult-project/catapult.git@polymer10-migration
Patch Set: Merge polymer10-migration int polymer10-merge Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: telemetry/telemetry/internal/platform/android_platform_backend.py
diff --git a/telemetry/telemetry/internal/platform/android_platform_backend.py b/telemetry/telemetry/internal/platform/android_platform_backend.py
index ab9ece8482aea6a48e3b02b39f5ffc23392d10d9..7399ead31b6a3ec4634e10e2bdb2fca0a3772151 100644
--- a/telemetry/telemetry/internal/platform/android_platform_backend.py
+++ b/telemetry/telemetry/internal/platform/android_platform_backend.py
@@ -47,6 +47,10 @@ except Exception:
surface_stats_collector = None
+_ARCH_TO_STACK_TOOL_ARCH = {
+ 'armeabi-v7a': 'arm',
+ 'arm64-v8a': 'arm64',
+}
_DEVICE_COPY_SCRIPT_FILE = os.path.abspath(os.path.join(
os.path.dirname(__file__), 'efficient_android_directory_copy.sh'))
_DEVICE_COPY_SCRIPT_LOCATION = (
@@ -515,7 +519,8 @@ class AndroidPlatformBackend(
logging.warning('Test certificate authority is already installed.')
return
self._device_cert_util = adb_install_cert.AndroidCertInstaller(
- self._device.adb.GetDeviceSerial(), None, ca_cert_path)
+ self._device.adb.GetDeviceSerial(), None, ca_cert_path,
+ adb_path=self._device.adb.GetAdbPath())
self._device_cert_util.install_cert(overwrite_cert=True)
def RemoveTestCa(self):
@@ -649,10 +654,11 @@ class AndroidPlatformBackend(
number_of_lines: Number of lines of log to return.
"""
return '\n'.join(self._device.RunShellCommand(
- ['logcat', '-d', '-t', str(number_of_lines)]))
+ ['logcat', '-d', '-t', str(number_of_lines)],
+ check_return=True, large_output=True))
def GetStandardOutput(self):
- return None
+ return 'Cannot get standard output on Android'
def GetStackTrace(self):
"""Returns stack trace.
@@ -674,7 +680,9 @@ class AndroidPlatformBackend(
# Try to symbolize logcat.
if os.path.exists(stack):
cmd = [stack]
- cmd.append('--arch=%s' % self.GetArchName())
+ arch = self.GetArchName()
+ arch = _ARCH_TO_STACK_TOOL_ARCH.get(arch, arch)
+ cmd.append('--arch=%s' % arch)
p = subprocess.Popen(cmd, stdin=subprocess.PIPE, stdout=subprocess.PIPE)
ret += Decorate('Stack from Logcat', p.communicate(input=logcat)[0])
@@ -688,6 +696,9 @@ class AndroidPlatformBackend(
stdout=subprocess.PIPE).communicate()[0])
return (True, ret)
+ def GetMinidumpPath(self):
+ return None
+
def IsScreenOn(self):
"""Determines if device screen is on."""
return self._device.IsScreenOn()

Powered by Google App Engine
This is Rietveld 408576698