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

Unified Diff: tools/telemetry/telemetry/core/browser_finder.py

Issue 902763002: Parallelize telemetry unit tests on Android bots (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove fyi bot config (will create another CL for it) Created 5 years, 10 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
Index: tools/telemetry/telemetry/core/browser_finder.py
diff --git a/tools/telemetry/telemetry/core/browser_finder.py b/tools/telemetry/telemetry/core/browser_finder.py
index f98317b38cee6287a0376c04d06403e4f07dceaf..b863d6e351438519970535af08d67e838964f918 100644
--- a/tools/telemetry/telemetry/core/browser_finder.py
+++ b/tools/telemetry/telemetry/core/browser_finder.py
@@ -61,12 +61,7 @@ def FindBrowser(options):
raise browser_finder_exceptions.BrowserFinderException(
'--remote requires --browser=cros-chrome or cros-chrome-guest.')
- devices = []
- if options.device and options.device != 'list':
- devices = device_finder.GetSpecifiedDevices(options)
- else:
- devices = device_finder.GetAllAvailableDevices(options)
-
+ devices = device_finder.GetDevicesMatchingOptions(options)
browsers = []
default_browsers = []
for device in devices:
@@ -168,7 +163,7 @@ def GetAllAvailableBrowserTypes(options):
Raises:
BrowserFinderException: Options are improperly set, or an error occurred.
"""
- devices = device_finder.GetAllAvailableDevices(options)
+ devices = device_finder.GetDevicesMatchingOptions(options)
possible_browsers = []
for device in devices:
possible_browsers.extend(GetAllAvailableBrowsers(options, device))

Powered by Google App Engine
This is Rietveld 408576698