commit b12e2ee0fb8d54e963d513c6da808a98ab5fa518
author Dean Liao <deanliao@chromium.org> Tue Nov 19 16:49:12 2013 +0800
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Wed Nov 20 07:46:41 2013 +0000
tree c622f25ce59ff1e03c61e909f40a5b5bddff3e7a
parent a6bc50e67bea184ee08cfa090ddf5c37ca4ff0e5

Accept extra_browser_args when initializing Chrome object.

Sometimes tests need to set extra_browser_args to use specific Telemetry
method. It provides an interface for users to do so.

BUG=None
TEST=manual

Change-Id: Ib2a7f89d93f6dd35064d469faeb621d788415688
Reviewed-on: https://chromium-review.googlesource.com/177321
Reviewed-by: Dean Liao <deanliao@chromium.org>
Commit-Queue: Dean Liao <deanliao@chromium.org>
Tested-by: Dean Liao <deanliao@chromium.org>

client/common_lib/cros/chrome.py

diff --git a/client/common_lib/cros/chrome.py b/client/common_lib/cros/chrome.py
index 35c642f..045cdce 100644
--- a/client/common_lib/cros/chrome.py
+++ b/client/common_lib/cros/chrome.py
@@ -21,7 +21,11 @@
 
 
     def __init__(self, logged_in=True, extension_paths=[], autotest_ext=False,
-                 num_tries=1):
+                 num_tries=1, extra_browser_args=None):
+        """
+        @param extra_browser_args: Additional argument(s) to pass to the
+            browser. It can be a string or a list.
+        """
         self._autotest_ext_path = None
         if autotest_ext:
             self._autotest_ext_path = os.path.join(os.path.dirname(__file__),
@@ -32,6 +36,9 @@
         self._browser_type = (self.BROWSER_TYPE_LOGIN
                 if logged_in else self.BROWSER_TYPE_GUEST)
         finder_options.browser_type = self.browser_type
+        if extra_browser_args:
+            finder_options.browser_options.AppendExtraBrowserArgs(
+                    extra_browser_args)
 
         if logged_in:
             extensions_to_load = finder_options.extensions_to_load