Also do not change RemoteWebDriver interface (it would be an BC break…

… as well)
lireincore · Jan 13, 2017 · cd4f352 · cd4f352
1 parent 4066768
commit cd4f352
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/Remote/RemoteWebDriver.php b/lib/Remote/RemoteWebDriver.php
@@ -20,6 +20,7 @@
 use Facebook\WebDriver\WebDriver;
 use Facebook\WebDriver\WebDriverBy;
 use Facebook\WebDriver\WebDriverCapabilities;
+use Facebook\WebDriver\WebDriverCommandExecutor;
 use Facebook\WebDriver\WebDriverElement;
 use Facebook\WebDriver\WebDriverHasInputDevices;
 use Facebook\WebDriver\WebDriverNavigation;
@@ -507,10 +508,10 @@ protected function newElement($id)
      * @deprecated To be removed in the future. Executor should be passed in the constructor.
      * @internal
      * @codeCoverageIgnore
-     * @param HttpCommandExecutor $executor
+     * @param WebDriverCommandExecutor $executor Despite the typehint, it have be an instance of HttpCommandExecutor.
      * @return RemoteWebDriver
      */
-    public function setCommandExecutor(HttpCommandExecutor $executor)
+    public function setCommandExecutor(WebDriverCommandExecutor $executor)
     {
         $this->executor = $executor;