Chromium Code Reviews| Index: tools/telemetry/telemetry/testing/run_tests.py |
| diff --git a/tools/telemetry/telemetry/testing/run_tests.py b/tools/telemetry/telemetry/testing/run_tests.py |
| index d460c891912554b07e053265075296d1368b247b..b266b55152e4d2e2a09ae25f00791b84de3801b6 100644 |
| --- a/tools/telemetry/telemetry/testing/run_tests.py |
| +++ b/tools/telemetry/telemetry/testing/run_tests.py |
| @@ -9,6 +9,7 @@ from telemetry.internal.browser import browser_finder |
| from telemetry.internal.browser import browser_finder_exceptions |
| from telemetry.internal.browser import browser_options |
| from telemetry.internal.platform import device_finder |
| +from telemetry.internal.util import binary_manager |
| from telemetry.internal.util import command_line |
| from telemetry.testing import browser_test_case |
| from telemetry.testing import options_for_unittests |
| @@ -45,6 +46,7 @@ class RunTestsCommand(command_line.OptparseCommand): |
| parser.add_option('--exact-test-filter', action='store_true', default=False, |
| help='Treat test filter as exact matches (default is ' |
| 'substring matches).') |
| + parser.add_option('--client-config', dest='client_config', default=None) |
| typ.ArgumentParser.add_option_group(parser, |
| "Options for running the tests", |
| @@ -61,6 +63,8 @@ class RunTestsCommand(command_line.OptparseCommand): |
| if not args.retry_limit and not args.positional_args: |
| args.retry_limit = 3 |
| + binary_manager.InitDependencyManager(args.client_config) |
|
nednguyen
2015/08/13 22:20:45
I think it's better to put this in the Run(self, a
aiolos (Not reviewing)
2015/08/13 22:28:32
We can only do this if we move the check for possi
|
| + |
| try: |
| possible_browser = browser_finder.FindBrowser(args) |
| except browser_finder_exceptions.BrowserFinderException, ex: |
| @@ -168,6 +172,9 @@ def _MatchesSelectedTest(name, selected_tests, selected_tests_are_exact): |
| def _SetUpProcess(child, context): # pylint: disable=W0613 |
|
nednguyen
2015/08/13 22:20:45
Nit: add comment explaining why we need to init th
aiolos (Not reviewing)
2015/08/13 22:28:32
Acknowledged.
|
| + if binary_manager.NeedsInit(): |
| + binary_manager.InitDependencyManager(context.client_config) |
| + |
| args = context |
| if args.device and args.device == 'android': |
| android_devices = device_finder.GetDevicesMatchingOptions(args) |