fixup! Add support for different WebDriver servers

w3c · Dec 5, 2014 · 682290c · 682290c
1 parent f28ea92
commit 682290c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/wptrunner/browsers/chrome.py b/wptrunner/browsers/chrome.py
@@ -50,7 +50,7 @@ class ChromeBrowser(Browser):
     """Chrome is backed by chromedriver, which is supplied through
     ``browsers.webdriver.ChromedriverLocalServer``."""
 
-    def __init__(self, logger, binary, webdriver_binary=None):
+    def __init__(self, logger, binary, webdriver_binary="chromedriver"):
         """Creates a new representation of Chrome.  The `binary` argument gives
         the browser binary to use for testing."""
         Browser.__init__(self, logger)

diff --git a/wptrunner/browsers/webdriver.py b/wptrunner/browsers/webdriver.py
@@ -99,8 +99,8 @@ def stop(self):
 class ChromedriverLocalServer(LocalServer):
     default_endpoint = "/wd/hub"
 
-    def __init__(self, logger, binary=None, port=None, endpoint=None):
-        LocalServer.__init__(self, logger, binary or "chromedriver", port=port, endpoint=endpoint)
+    def __init__(self, logger, binary="chromedriver", port=None, endpoint=None):
+        LocalServer.__init__(self, logger, binary, port=port, endpoint=endpoint)
         # TODO: verbose logging
         self.cmd = [self.binary,
                     cmd_arg("port", str(self.port)) if self.port else "",