Index: tools/telemetry/telemetry/core/backends/adb_commands.py |
diff --git a/tools/telemetry/telemetry/core/backends/adb_commands.py b/tools/telemetry/telemetry/core/backends/adb_commands.py |
index be822c2d3ca773ffe682c800be005035e1d59dd6..e02a3faf241a5c14696110a5849975290bdd28b3 100644 |
--- a/tools/telemetry/telemetry/core/backends/adb_commands.py |
+++ b/tools/telemetry/telemetry/core/backends/adb_commands.py |
@@ -14,7 +14,7 @@ from telemetry.core import util |
util.AddDirToPythonPath(util.GetChromiumSrcDir(), 'build', 'android') |
try: |
from pylib import android_commands # pylint: disable=F0401 |
- from pylib import cmd_helper # pylint: disable=F0401 |
+ from pylib import constants # pylint: disable=F0401 |
from pylib import forwarder # pylint: disable=F0401 |
from pylib import ports # pylint: disable=F0401 |
except Exception: |
@@ -41,10 +41,6 @@ def ResetTestServerPortAllocation(): |
return ports.ResetTestServerPortAllocation() |
-def GetOutDirectory(): |
- return cmd_helper.OutDirectory.get() |
- |
- |
class AdbCommands(object): |
"""A thin wrapper around ADB""" |
@@ -151,10 +147,11 @@ def HasForwarder(buildtype=None): |
if not buildtype: |
return (HasForwarder(buildtype='Release') or |
HasForwarder(buildtype='Debug')) |
- return (os.path.exists(os.path.join(GetOutDirectory(), buildtype, |
- 'device_forwarder')) and |
- os.path.exists(os.path.join(GetOutDirectory(), buildtype, |
- 'host_forwarder'))) |
+ device_forwarder = os.path.join( |
+ constants.GetOutDirectory(build_type=buildtype), 'device_forwarder') |
+ host_forwarder = os.path.join( |
+ constants.GetOutDirectory(build_type=buildtype), 'host_forwarder') |
+ return os.path.exists(device_forwarder) and os.path.exists(host_forwarder) |
class Forwarder(object): |
def __init__(self, adb, *port_pairs): |
@@ -165,13 +162,14 @@ class Forwarder(object): |
for port_pair in port_pairs] |
self._port_pairs = new_port_pairs |
+ if HasForwarder('Release'): |
+ constants.SetBuildType('Release') |
+ elif HasForwarder('Debug'): |
+ constants.SetBuildType('Debug') |
+ else: |
+ raise Exception('Build forwarder2') |
forwarder.Forwarder.Map(new_port_pairs, self._adb) |
- @staticmethod |
- def _GetBuildType(): |
- assert HasForwarder() |
- return 'Debug' if HasForwarder('Debug') else 'Release' |
- |
@property |
def url(self): |
return 'http://localhost:%i' % self._host_port |