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

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

Issue 23512005: BrowserOptions.extra_browser_args is now a set. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 3 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 | « tools/perf/metrics/v8_object_stats.py ('k') | tools/telemetry/telemetry/core/browser_unittest.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 0b490bb9713b2e43e55e6f0f422d3f2a278d6bd9..82df93016454ca24d8109e01485e8d207ab418c7 100644
--- a/tools/telemetry/telemetry/core/browser_options.py
+++ b/tools/telemetry/telemetry/core/browser_options.py
@@ -33,7 +33,7 @@ class BrowserFinderOptions(optparse.Values):
self.dont_override_profile = False
self.profile_dir = None
self.profile_type = None
- self.extra_browser_args = []
+ self._extra_browser_args = set()
self.extra_wpr_args = []
self.show_stdout = False
self.extensions_to_load = []
@@ -214,7 +214,7 @@ class BrowserFinderOptions(optparse.Values):
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)
+ self._extra_browser_args.update(tmp)
delattr(self, 'extra_browser_args_as_string')
if self.extra_wpr_args_as_string: # pylint: disable=E1101
tmp = shlex.split(
@@ -238,14 +238,20 @@ class BrowserFinderOptions(optparse.Values):
parser.parse_args = ParseArgs
return parser
- def AppendExtraBrowserArg(self, arg):
- if arg not in self.extra_browser_args:
- self.extra_browser_args.append(arg)
+ def AppendExtraBrowserArgs(self, args):
+ if isinstance(args, list):
+ self._extra_browser_args.update(args)
+ else:
+ self._extra_browser_args.add(args)
def MergeDefaultValues(self, defaults):
for k, v in defaults.__dict__.items():
self.ensure_value(k, v)
+ @property
+ def extra_browser_args(self):
+ return self._extra_browser_args
+
class BrowserOptions():
"""Options to be used for launching a browser."""
def __init__(self):
« no previous file with comments | « tools/perf/metrics/v8_object_stats.py ('k') | tools/telemetry/telemetry/core/browser_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698