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

Unified Diff: tools/chrome_remote_control/chrome_remote_control/browser_options.py

Issue 11361165: [chrome_remote_control] Rename chrome_remote_control to telemetry. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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/chrome_remote_control/chrome_remote_control/browser_options.py
diff --git a/tools/chrome_remote_control/chrome_remote_control/browser_options.py b/tools/chrome_remote_control/chrome_remote_control/browser_options.py
deleted file mode 100644
index d624c85b73df5982ec7408bfa6bbca2d291a010e..0000000000000000000000000000000000000000
--- a/tools/chrome_remote_control/chrome_remote_control/browser_options.py
+++ /dev/null
@@ -1,148 +0,0 @@
-# Copyright (c) 2012 The Chromium Authors. All rights reserved.
-# Use of this source code is governed by a BSD-style license that can be
-# found in the LICENSE file.
-import optparse
-import os
-import sys
-import shlex
-import logging
-
-from chrome_remote_control import browser_finder
-from chrome_remote_control import wpr_modes
-
-class BrowserOptions(optparse.Values):
- """Options to be used for discovering and launching a browser."""
-
- def __init__(self, browser_type=None):
- optparse.Values.__init__(self)
-
- self.browser_type = browser_type
- self.browser_executable = None
- self.chrome_root = None
- self.android_device = None
- self.cros_ssh_identity = None
-
- self.dont_override_profile = False
- self.extra_browser_args = []
- self.show_stdout = False
-
- self.cros_remote = None
- self.wpr_mode = wpr_modes.WPR_OFF
- self.wpr_make_javascript_deterministic = True
-
- self.verbosity = 0
-
- def Copy(self):
- other = BrowserOptions()
- other.__dict__.update(self.__dict__)
- return other
-
- def CreateParser(self, *args, **kwargs):
- parser = optparse.OptionParser(*args, **kwargs)
-
- # Selection group
- group = optparse.OptionGroup(parser, 'Which browser to use')
- group.add_option('--browser',
- dest='browser_type',
- default=None,
- help='Browser type to run, '
- 'in order of priority. Supported values: list,%s' %
- browser_finder.ALL_BROWSER_TYPES)
- group.add_option('--browser-executable',
- dest='browser_executable',
- help='The exact browser to run.')
- group.add_option('--chrome-root',
- dest='chrome_root',
- help='Where to look for chrome builds.'
- 'Defaults to searching parent dirs by default.')
- group.add_option('--device',
- dest='android_device',
- help='The android device ID to use'
- 'If not specified, only 0 or 1 connected devcies are supported.')
- group.add_option(
- '--remote',
- dest='cros_remote',
- default=os.getenv('REMOTE'),
- help='The IP address of a remote ChromeOS device to use. ' +
- 'Defaults to $REMOTE from environment variable if set.')
- group.add_option('--identity',
- dest='cros_ssh_identity',
- default=None,
- help='The identity file to use when ssh\'ing into the ChromeOS device')
- parser.add_option_group(group)
-
- # Browser options
- group = optparse.OptionGroup(parser, 'Browser options')
- group.add_option('--dont-override-profile', action='store_true',
- dest='dont_override_profile',
- help='Uses the regular user profile instead of a clean one')
- group.add_option('--extra-browser-args',
- dest='extra_browser_args_as_string',
- help='Additional arguments to pass to the browser when it starts')
- group.add_option('--show-stdout',
- action='store_true',
- help='When possible, will display the stdout of the process')
- parser.add_option_group(group)
-
- # Page set options
- group = optparse.OptionGroup(parser, 'Page set options')
- group.add_option('--record', action='store_const',
- dest='wpr_mode', const=wpr_modes.WPR_RECORD,
- help='Record to the page set archive')
- group.add_option('--page-repeat', dest='page_repeat', default=1,
- help='Number of times to repeat each individual ' +
- 'page in the pageset before proceeding.')
- group.add_option('--pageset-repeat', dest='pageset_repeat', default=1,
- help='Number of times to repeat the entire pageset ' +
- 'before finishing.')
- group.add_option('--test-shuffle', action='store_true', dest='test_shuffle',
- help='Shuffle the order of pages within a pageset.')
- group.add_option('--test-shuffle-order-file',
- dest='test_shuffle_order_file', default=None,
- help='Filename of an output of a previously run test on the current ' +
- 'pageset. The tests will run in the same order again, overriding ' +
- 'what is specified by --page-repeat and --pageset-repeat.')
- parser.add_option_group(group)
-
- # Debugging options
- group = optparse.OptionGroup(parser, 'When things go wrong')
- group.add_option(
- '-v', '--verbose', action='count', dest='verbosity',
- help='Increase verbosity level (repeat as needed)')
- parser.add_option_group(group)
-
- real_parse = parser.parse_args
- def ParseArgs(args=None):
- defaults = parser.get_default_values()
- for k, v in defaults.__dict__.items():
- if k in self.__dict__ and self.__dict__[k] != None:
- continue
- self.__dict__[k] = v
- ret = real_parse(args, self) # pylint: disable=E1121
-
- if self.verbosity >= 2:
- logging.basicConfig(level=logging.DEBUG)
- elif self.verbosity:
- logging.basicConfig(level=logging.INFO)
- else:
- logging.basicConfig(level=logging.WARNING)
-
- if self.browser_executable and not self.browser_type:
- self.browser_type = 'exact'
- if not self.browser_executable and not self.browser_type:
- sys.stderr.write('Must provide --browser=<type>. ' +
- 'Use --browser=list for valid options.\n')
- sys.exit(1)
- if self.browser_type == 'list':
- types = browser_finder.GetAllAvailableBrowserTypes(self)
- sys.stderr.write('Available browsers:\n')
- sys.stdout.write(' %s\n' % '\n '.join(types))
- sys.exit(1)
- if self.extra_browser_args_as_string: # pylint: disable=E1101
- tmp = shlex.split(
- self.extra_browser_args_as_string) # pylint: disable=E1101
- self.extra_browser_args.extend(tmp)
- delattr(self, 'extra_browser_args_as_string')
- return ret
- parser.parse_args = ParseArgs
- return parser

Powered by Google App Engine
This is Rietveld 408576698