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

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

Issue 68263025: Android: removes old "keep_test_server_ports". (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: comments Created 7 years, 1 month 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_options.py
diff --git a/tools/telemetry/telemetry/core/browser_options.py b/tools/telemetry/telemetry/core/browser_options.py
index c83491f89e1b4218f89f9f1f570419c952dcf8cb..ec05a404b9c6d671c0bc9615b23b83123120e23d 100644
--- a/tools/telemetry/telemetry/core/browser_options.py
+++ b/tools/telemetry/telemetry/core/browser_options.py
@@ -216,8 +216,6 @@ class BrowserOptions(object):
self.clear_sytem_cache_for_browser_and_profile_on_start = False
self.startup_url = None
- self.keep_test_server_ports = False
-
# Background pages of built-in component extensions can interfere with
# performance measurements.
self.disable_component_extensions_with_background_pages = True
@@ -248,15 +246,6 @@ class BrowserOptions(object):
help='When possible, will display the stdout of the process')
parser.add_option_group(group)
- # Android options. TODO(achuith): Move to AndroidBrowserOptions.
- group = optparse.OptionGroup(parser, 'Android options')
- group.add_option('--keep_test_server_ports',
- action='store_true',
- help='Indicates the test server ports must be kept. When this is run '
- 'via a sharder the test server ports should be kept and should '
- 'not be reset.')
- parser.add_option_group(group)
-
group = optparse.OptionGroup(parser, 'Compatibility options')
group.add_option('--gtest_output',
help='Ignored argument for compatibility with runtest.py harness')

Powered by Google App Engine
This is Rietveld 408576698