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

Unified Diff: build/android/run_tests.py

Issue 10790006: clean use_emulator option for inconsistence issue (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix for bulach's comments Created 8 years, 5 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/run_tests.py
diff --git a/build/android/run_tests.py b/build/android/run_tests.py
index f749aaec8d92b035250e297da1ff3f83eec9c071..d98bac788a87ddb66a4dbf47c13dad2a289307b7 100755
--- a/build/android/run_tests.py
+++ b/build/android/run_tests.py
@@ -323,7 +323,7 @@ def _RunATestSuite(options):
buildbot_emulators = []
if options.use_emulator:
- for n in range(options.use_emulator):
+ for n in range(options.emulator_count):
t = TimeProfile('Emulator launch %d' % n)
buildbot_emulator = emulator.Emulator(options.fast_and_loose)
buildbot_emulator.Launch(kill_all_emulators=n == 0)
@@ -384,7 +384,7 @@ def _RunATestSuite(options):
def Dispatch(options):
"""Dispatches the tests, sharding if possible.
- If options.use_emulator is True, all tests will be run in a new emulator
+ If options.use_emulator is True, all tests will be run in new emulator
instance.
Args:
@@ -447,9 +447,12 @@ def main(argv):
'subfolder debug_info_dumps under the same directory'
'in where the test_suite exists.')
option_parser.add_option('-e', '--emulator', dest='use_emulator',
- help='Run tests in a new instance of emulator',
- type='int',
- default=0)
+ action='store_true',
+ help='Run tests in a new instance of emulator')
+ option_parser.add_option('-n', '--emulator_count',
+ type='int', default=1,
+ help='Number of emulators to launch for running the '
+ 'tests.')
option_parser.add_option('-x', '--xvfb', dest='use_xvfb',
action='store_true',
help='Use Xvfb around tests (ignored if not Linux)')
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698