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

Side by Side Diff: infra/recipes/android/perf.py

Issue 2434403002: Revert of [Android] Always use devil's adb in the perf recipes. (RELAND) (Closed)
Patch Set: Created 4 years, 2 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 unified diff | Download patch
« no previous file with comments | « no previous file | infra/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__1_.json » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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 from recipe_engine.types import freeze 5 from recipe_engine.types import freeze
6 6
7 7
8 DEPS = [ 8 DEPS = [
9 'build/adb', 9 'build/adb',
10 'build/chromium', 10 'build/chromium',
(...skipping 138 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 api.chromium_tests.get_compile_targets_for_scripts().json.output 149 api.chromium_tests.get_compile_targets_for_scripts().json.output
150 150
151 builder['tests'] = api.chromium_tests.generate_tests_from_test_spec( 151 builder['tests'] = api.chromium_tests.generate_tests_from_test_spec(
152 api, test_spec, builder, buildername, mastername, False, None, 152 api, test_spec, builder, buildername, mastername, False, None,
153 scripts_compile_targets, [api.chromium_tests.steps.generate_script], 153 scripts_compile_targets, [api.chromium_tests.steps.generate_script],
154 bot_update_step) 154 bot_update_step)
155 155
156 api.path['checkout'] = api.path['slave_build'].join('src') 156 api.path['checkout'] = api.path['slave_build'].join('src')
157 api.chromium_android.clean_local_files() 157 api.chromium_android.clean_local_files()
158 158
159 # TODO(jbudorick): Remove this after resolving
160 # https://github.com/catapult-project/catapult/issues/2901
161 devil_path = api.path['checkout'].join('third_party', 'catapult', 'devil')
162 api.python.inline(
163 'initialize devil',
164 """
165 import sys
166 sys.path.append(sys.argv[1])
167 from devil import devil_env
168 devil_env.config.Initialize()
169 devil_env.config.PrefetchPaths(dependencies=['adb'])
170 """,
171 args=[devil_path])
172 api.adb.set_adb_path(
173 devil_path.join('bin', 'deps', 'linux2', 'x86_64', 'bin', 'adb'))
174
175 api.chromium_android.download_build(bucket=builder['bucket'], 159 api.chromium_android.download_build(bucket=builder['bucket'],
176 path=builder['path'](api)) 160 path=builder['path'](api))
177 161
178 api.chromium_android.common_tests_setup_steps( 162 api.chromium_android.common_tests_setup_steps(
179 perf_setup=True, 163 perf_setup=True,
180 remove_system_webview=builder.get('remove_system_webview', False)) 164 remove_system_webview=builder.get('remove_system_webview', False))
181 165
182 required_apks = builder.get('required_apks', []) 166 required_apks = builder.get('required_apks', [])
183 for apk in required_apks: 167 for apk in required_apks:
184 api.chromium_android.adb_install_apk(apk) 168 api.chromium_android.adb_install_apk(apk)
(...skipping 182 matching lines...) Expand 10 before | Expand all | Expand 10 after
367 "adb_status": "device", 351 "adb_status": "device",
368 "imei_slice": "", 352 "imei_slice": "",
369 "ro.build.product": "bullhead", 353 "ro.build.product": "bullhead",
370 "ro.build.id": "MDB08Q", 354 "ro.build.id": "MDB08Q",
371 "serial": "00d0d567893340f4", 355 "serial": "00d0d567893340f4",
372 "wifi_ip": "" 356 "wifi_ip": ""
373 }] 357 }]
374 }}), 358 }}),
375 retcode=87)) 359 retcode=87))
376 360
OLDNEW
« no previous file with comments | « no previous file | infra/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__1_.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698