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

Side by Side Diff: mojo/devtools/common/devtoolslib/shell_config.py

Issue 1437383002: Never try to use root to run the mojo tools. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 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 unified diff | Download patch
« no previous file with comments | « mojo/devtools/common/devtoolslib/shell_arguments.py ('k') | mojo/devtools/common/mojo_test » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 """Configuration for the shell abstraction. 5 """Configuration for the shell abstraction.
6 6
7 This module declares ShellConfig and knows how to compute it from command-line 7 This module declares ShellConfig and knows how to compute it from command-line
8 arguments, applying any default paths inferred from the checkout, configuration 8 arguments, applying any default paths inferred from the checkout, configuration
9 file, etc. 9 file, etc.
10 """ 10 """
(...skipping 19 matching lines...) Expand all
30 self.map_origin_list = [] 30 self.map_origin_list = []
31 self.dev_servers = [] 31 self.dev_servers = []
32 self.reuse_servers = False 32 self.reuse_servers = False
33 self.content_handlers = dict() 33 self.content_handlers = dict()
34 self.verbose = None 34 self.verbose = None
35 35
36 # Android-only. 36 # Android-only.
37 self.adb_path = None 37 self.adb_path = None
38 self.target_device = None 38 self.target_device = None
39 self.logcat_tags = None 39 self.logcat_tags = None
40 self.require_root = False
41 40
42 # Desktop-only. 41 # Desktop-only.
43 self.use_osmesa = None 42 self.use_osmesa = None
44 43
45 44
46 class DevServerConfig(object): 45 class DevServerConfig(object):
47 """Configuration for a development server running on a host and available to 46 """Configuration for a development server running on a host and available to
48 the shell. 47 the shell.
49 """ 48 """
50 def __init__(self): 49 def __init__(self):
(...skipping 161 matching lines...) Expand 10 before | Expand all | Expand 10 after
212 211
213 if 'content_handlers' in config: 212 if 'content_handlers' in config:
214 try: 213 try:
215 for (mime_type, 214 for (mime_type,
216 content_handler_url) in config['content_handlers'].iteritems(): 215 content_handler_url) in config['content_handlers'].iteritems():
217 shell_config.content_handlers[mime_type] = content_handler_url 216 shell_config.content_handlers[mime_type] = content_handler_url
218 except (ValueError, KeyError): 217 except (ValueError, KeyError):
219 raise ShellConfigurationException('Failed to parse content_handlers in ' 218 raise ShellConfigurationException('Failed to parse content_handlers in '
220 'the config file.') 219 'the config file.')
221 return shell_config 220 return shell_config
OLDNEW
« no previous file with comments | « mojo/devtools/common/devtoolslib/shell_arguments.py ('k') | mojo/devtools/common/mojo_test » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698