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

Side by Side Diff: scripts/slave/recipe_modules/chromium/api.py

Issue 2485383007: Make perf android tests use devil adb (Closed)
Patch Set: ~ Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import re 5 import re
6 6
7 from recipe_engine import recipe_api 7 from recipe_engine import recipe_api
8 from recipe_engine import util as recipe_util 8 from recipe_engine import util as recipe_util
9 9
10 class TestLauncherFilterFileInputPlaceholder(recipe_util.InputPlaceholder): 10 class TestLauncherFilterFileInputPlaceholder(recipe_util.InputPlaceholder):
(...skipping 852 matching lines...) Expand 10 before | Expand all | Expand 10 after
863 if mode: 863 if mode:
864 args.extend(['--mode', mode]) 864 args.extend(['--mode', mode])
865 self.m.python( 865 self.m.python(
866 step_name, 866 step_name,
867 self.package_repo_resource( 867 self.package_repo_resource(
868 'scripts', 'slave', 'chromium', 'archive_build.py'), 868 'scripts', 'slave', 'chromium', 'archive_build.py'),
869 args, 869 args,
870 infra_step=True, 870 infra_step=True,
871 **kwargs) 871 **kwargs)
872 872
873 def list_perf_tests(self, browser, num_shards, device=None): 873 def list_perf_tests(self, browser, num_shards, device=None,
874 pass_adb_path=False):
874 args = ['list', '--browser', browser, '--json-output', 875 args = ['list', '--browser', browser, '--json-output',
875 self.m.json.output(), '--num-shards', num_shards] 876 self.m.json.output(), '--num-shards', num_shards]
876 if device: 877 if device:
877 args += ['--device', device] 878 args += ['--device', device]
878 879
880 if pass_adb_path:
881 args.extend(['--adb-path', self.m.adb.adb_path()])
jbudorick 2016/11/11 22:47:57 This shouldn't be here. run_benchmark doesn't supp
882
879 return self.m.python( 883 return self.m.python(
880 'List Perf Tests', 884 'List Perf Tests',
881 self.m.path['checkout'].join('tools', 'perf', 'run_benchmark'), 885 self.m.path['checkout'].join('tools', 'perf', 'run_benchmark'),
882 args, 886 args,
883 step_test_data=lambda: self.m.json.test_api.output({ 887 step_test_data=lambda: self.m.json.test_api.output({
884 "steps": { 888 "steps": {
885 "blink_perf.all.release": { 889 "blink_perf.all.release": {
886 "cmd": "/usr/bin/python /path/to/run_benchmark --a=1 -v --b=2", 890 "cmd": "/usr/bin/python /path/to/run_benchmark --a=1 -v --b=2",
887 "perf_dashboard_id": "blink_perf.all", 891 "perf_dashboard_id": "blink_perf.all",
888 "device_affinity": 0 892 "device_affinity": 0
(...skipping 22 matching lines...) Expand all
911 })) 915 }))
912 916
913 def get_annotate_by_test_name(self, test_name): 917 def get_annotate_by_test_name(self, test_name):
914 return 'graphing' 918 return 'graphing'
915 919
916 def download_lto_plugin(self): 920 def download_lto_plugin(self):
917 return self.m.python( 921 return self.m.python(
918 name='download LTO plugin', 922 name='download LTO plugin',
919 script=self.m.path['checkout'].join( 923 script=self.m.path['checkout'].join(
920 'build', 'download_gold_plugin.py')) 924 'build', 'download_gold_plugin.py'))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698