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

Unified Diff: build/android/test_runner.py

Issue 22622004: [Android] Removes unused and deprecated flags in test_runner.py (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removes other deprecated options Created 7 years, 4 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 | « build/android/pylib/uiautomator/test_runner.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/test_runner.py
diff --git a/build/android/test_runner.py b/build/android/test_runner.py
index 4113158fd38235779b852788867da6071b183cc3..b8de366a40ec794276199c868280ee4abab335cc 100755
--- a/build/android/test_runner.py
+++ b/build/android/test_runner.py
@@ -68,13 +68,6 @@ def AddCommonOptions(option_parser):
default=0,
action='count',
help='Verbose level (multiple times for more)')
- profilers = ['devicestatsmonitor', 'chrometrace', 'dumpheap', 'smaps',
- 'traceview']
- option_parser.add_option('--profiler', dest='profilers', action='append',
- choices=profilers,
- help=('Profiling tool to run during test. Pass '
- 'multiple times to run multiple profilers. '
- 'Available profilers: %s' % profilers))
option_parser.add_option('--tool',
dest='tool',
help=('Run the test under a tool '
@@ -172,10 +165,6 @@ def AddJavaTestOptions(option_parser):
'kept. When this is run via a sharder '
'the test server ports should be kept and '
'should not be reset.'))
- # TODO(gkanwar): This option is deprecated. Remove it in the future.
- option_parser.add_option('--disable_assertions', action='store_true',
- help=('(DEPRECATED) Run with java assertions '
- 'disabled.'))
option_parser.add_option('--test_data', action='append', default=[],
help=('Each instance defines a directory of test '
'data that should be copied to the target(s) '
@@ -228,9 +217,6 @@ def AddInstrumentationTestOptions(option_parser):
option_parser.add_option('-w', '--wait_debugger', dest='wait_for_debugger',
action='store_true',
help='Wait for debugger.')
- #TODO(craigdh): Remove option once -I is no longer passed downstream.
- option_parser.add_option('-I', dest='install_apk', action='store_true',
- help='(DEPRECATED) Install the test apk.')
option_parser.add_option(
'--test-apk', dest='test_apk',
help=('The name of the apk containing the tests '
@@ -293,7 +279,6 @@ def ProcessInstrumentationOptions(options, error_func):
options.test_data,
options.save_perf_json,
options.screenshot_failures,
- options.disable_assertions,
options.wait_for_debugger,
options.test_apk,
options.test_apk_path,
@@ -363,7 +348,6 @@ def ProcessUIAutomatorOptions(options, error_func):
options.test_data,
options.save_perf_json,
options.screenshot_failures,
- options.disable_assertions,
options.uiautomator_jar,
options.uiautomator_info_jar,
options.package_name)
« no previous file with comments | « build/android/pylib/uiautomator/test_runner.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698