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

Unified Diff: scripts/slave/recipe_modules/bisect_tester/perf_test.py

Issue 2045043002: Allow multiple devices on bisects hosts (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: solve coverage problem Created 4 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 | « scripts/slave/recipe_modules/bisect_tester/api.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipe_modules/bisect_tester/perf_test.py
diff --git a/scripts/slave/recipe_modules/bisect_tester/perf_test.py b/scripts/slave/recipe_modules/bisect_tester/perf_test.py
index 057e8a152aaf8a24f12059e0a802cff38b1a93cb..02acfab6eba03243fbee4abf9b566516831df83b 100644
--- a/scripts/slave/recipe_modules/bisect_tester/perf_test.py
+++ b/scripts/slave/recipe_modules/bisect_tester/perf_test.py
@@ -91,6 +91,12 @@ def run_perf_test(api, test_config, **kwargs):
use_chartjson = bool('chartjson' in command)
is_telemetry = _is_telemetry_command(command)
start_time = time.time()
+
+ if api.m.chromium.c.TARGET_PLATFORM == 'android' and is_telemetry:
+ device_serial_number = api.device_to_test;
+ if device_serial_number:
+ command += '--device ' + device_serial_number # pragma: no cover
RobertoCN 2016/06/22 21:34:40 shouldn't it be ' --device ' ?
Ziqi Xiong 2016/06/22 21:52:25 Nice catch, thanks!
+
for i in range(repeat_cnt):
elapsed_minutes = (time.time() - start_time) / 60.0
# A limit of 0 means 'no timeout set'.
« no previous file with comments | « scripts/slave/recipe_modules/bisect_tester/api.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698