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

Unified Diff: tools/run_perf.py

Issue 1220743002: [android] Remove legacy architecture configs from perf runner. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 6 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/run_perf.py
diff --git a/tools/run_perf.py b/tools/run_perf.py
index b09d5b5ed596fcc53d453fe239368a071595eb7b..3f73ee6c250693bd48dd978e3b4bedd5eedeac1c 100755
--- a/tools/run_perf.py
+++ b/tools/run_perf.py
@@ -108,11 +108,7 @@ from testrunner.local import commands
from testrunner.local import utils
ARCH_GUESS = utils.DefaultArch()
-SUPPORTED_ARCHS = ["android_arm",
- "android_arm64",
- "android_ia32",
- "android_x64",
- "arm",
+SUPPORTED_ARCHS = ["arm",
"ia32",
"mips",
"mipsel",
@@ -470,7 +466,7 @@ class Platform(object):
@staticmethod
def GetPlatform(options):
- if options.arch.startswith("android"):
+ if options.android_build_tools:
return AndroidPlatform(options)
else:
return DesktopPlatform(options)
@@ -632,7 +628,8 @@ def Main(args):
logging.getLogger().setLevel(logging.INFO)
parser = optparse.OptionParser()
parser.add_option("--android-build-tools",
- help="Path to chromium's build/android.")
+ help="Path to chromium's build/android. Specifying this "
+ "option will run tests using android platform.")
parser.add_option("--arch",
help=("The architecture to run tests for, "
"'auto' or 'native' for auto-detect"),
@@ -663,15 +660,8 @@ def Main(args):
print "Unknown architecture %s" % options.arch
return 1
- if (bool(options.arch.startswith("android")) !=
- bool(options.android_build_tools)): # pragma: no cover
- print ("Android architectures imply setting --android-build-tools and the "
- "other way around.")
- return 1
-
- if (options.device and not
- options.arch.startswith("android")): # pragma: no cover
- print "Specifying a device requires an Android architecture to be used."
+ if (options.device and not options.android_build_tools): # pragma: no cover
+ print "Specifying a device requires Android build tools."
return 1
workspace = os.path.abspath(os.path.join(os.path.dirname(__file__), ".."))
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698