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

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

Issue 2442663003: [android] Resume passing adb path to the test runner in android/perf. (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 186 matching lines...) Expand 10 before | Expand all | Expand 10 after
197 except api.step.StepFailure as f: 197 except api.step.StepFailure as f:
198 failures.append(f) 198 failures.append(f)
199 199
200 dynamic_perf_tests = api.chromium_tests.steps.DynamicPerfTests( 200 dynamic_perf_tests = api.chromium_tests.steps.DynamicPerfTests(
201 builder['perf_id'], 'android', None, 201 builder['perf_id'], 'android', None,
202 max_battery_temp=builder.get('max_battery_temp'), 202 max_battery_temp=builder.get('max_battery_temp'),
203 num_device_shards=builder['num_device_shards'], 203 num_device_shards=builder['num_device_shards'],
204 num_host_shards=builder.get('num_host_shards', 1), 204 num_host_shards=builder.get('num_host_shards', 1),
205 shard_index=builder.get('shard_index', 0), 205 shard_index=builder.get('shard_index', 0),
206 override_browser_name=builder.get('browser_name'), 206 override_browser_name=builder.get('browser_name'),
207 enable_platform_mode=builder.get('enable_platform_mode'), 207 enable_platform_mode=builder.get('enable_platform_mode'))
208 pass_adb_path=False)
209 dynamic_perf_tests.run(api, None) 208 dynamic_perf_tests.run(api, None)
210 209
211 if failures: 210 if failures:
212 raise api.step.StepFailure('src-side perf tests failed %s' % failures) 211 raise api.step.StepFailure('src-side perf tests failed %s' % failures)
213 finally: 212 finally:
214 api.chromium_android.common_tests_final_steps( 213 api.chromium_android.common_tests_final_steps(
215 logcat_gs_bucket='chromium-android') 214 logcat_gs_bucket='chromium-android')
216 215
217 216
218 def _sanitize_nonalpha(text): 217 def _sanitize_nonalpha(text):
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
367 "adb_status": "device", 366 "adb_status": "device",
368 "imei_slice": "", 367 "imei_slice": "",
369 "ro.build.product": "bullhead", 368 "ro.build.product": "bullhead",
370 "ro.build.id": "MDB08Q", 369 "ro.build.id": "MDB08Q",
371 "serial": "00d0d567893340f4", 370 "serial": "00d0d567893340f4",
372 "wifi_ip": "" 371 "wifi_ip": ""
373 }] 372 }]
374 }}), 373 }}),
375 retcode=87)) 374 retcode=87))
376 375
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