Minor codestyle fixes

Author: OndraM

lib/Chrome/ChromeOptions.php

@@ -85,8 +85,7 @@ public function addExtensions(array $paths) {
   }
 
   /**
-   * @param array $encoded_extensions An array of base64 encoded of the
-   *                                  extensions.
+   * @param array $encoded_extensions An array of base64 encoded of the extensions.
    * @return ChromeOptions
    */
   public function addEncodedExtensions(array $encoded_extensions) {
@@ -109,8 +108,7 @@ public function setExperimentalOption($name, $value) {
   }
 
   /**
-   * @return DesiredCapabilities The DesiredCapabilities for Chrome with this
-   *                             options.
+   * @return DesiredCapabilities The DesiredCapabilities for Chrome with this options.
    */
   public function toCapabilities() {
     $capabilities = DesiredCapabilities::chrome();

lib/Firefox/FirefoxDriver.php

@@ -16,6 +16,9 @@
 namespace Facebook\WebDriver\Firefox;
 
 class FirefoxDriver {
-
   const PROFILE = 'firefox_profile';
+
+  private function __construct()
+  {
+  }
 }

lib/Firefox/FirefoxProfile.php

@@ -55,7 +55,7 @@ public function addExtension($extension) {
   /**
    * @param string $extension_datas The path to the folder containing the datas to add to the extension
    * @return FirefoxProfile
-   */  
+   */
   public function addExtensionDatas($extension_datas) {
     if (!is_dir($extension_datas)) {
       return;
@@ -73,7 +73,7 @@ public function setRdfFile($rdf_file) {
     if (!is_file($rdf_file)) {
       return;
     }
-    
+
     $this->rdf_file = $rdf_file;
     return $this;
   }

lib/Remote/DriverCommand.php

@@ -19,7 +19,6 @@
  * This list of command defined in the WebDriver json wire protocol.
  */
 class DriverCommand {
-
   const GET_ALL_SESSIONS = "getAllSessions";
   const GET_CAPABILITIES = "getCapabilities";
   const NEW_SESSION = "newSession";
@@ -126,16 +125,14 @@ class DriverCommand {
   const SET_SCREEN_ORIENTATION = "setScreenOrientation";
   const GET_SCREEN_ORIENTATION = "getScreenOrientation";
 
-  // These belong to the Advanced user interactions - an element is
-  // optional for these commands.
+  // These belong to the Advanced user interactions - an element is optional for these commands.
   const CLICK = "mouseClick";
   const DOUBLE_CLICK = "mouseDoubleClick";
   const MOUSE_DOWN = "mouseButtonDown";
   const MOUSE_UP = "mouseButtonUp";
   const MOVE_TO = "mouseMoveTo";
 
-  // Those allow interactions with the Input Methods installed on
-  // the system.
+  // Those allow interactions with the Input Methods installed on the system.
   const IME_GET_AVAILABLE_ENGINES = "imeGetAvailableEngines";
   const IME_GET_ACTIVE_ENGINE = "imeGetActiveEngine";
   const IME_IS_ACTIVATED = "imeIsActivated";
@@ -168,4 +165,7 @@ class DriverCommand {
   const GET_NETWORK_CONNECTION = "getNetworkConnection";
   const SET_NETWORK_CONNECTION = "setNetworkConnection";
 
+  private function __construct()
+  {
+  }
 }

lib/Remote/RemoteWebDriver.php

@@ -74,7 +74,7 @@ public static function create(
   ) {
     $url = preg_replace('#/+$#', '', $url);
 
-    // Passing DesiredCapabilities as $desired_capabilities is encourged but
+    // Passing DesiredCapabilities as $desired_capabilities is encouraged but
     // array is also accepted for legacy reason.
     if ($desired_capabilities instanceof DesiredCapabilities) {
       $desired_capabilities = $desired_capabilities->toArray();

lib/Remote/Service/DriverCommandExecutor.php

@@ -38,13 +38,12 @@ public function __construct(DriverService $service) {
 
   /**
    * @param WebDriverCommand $command
-   * @param array            $curl_opts
    *
    * @return mixed
    * @throws WebDriverException
    * @throws \Exception
    */
-  public function execute(WebDriverCommand $command, $curl_opts = array()) {
+  public function execute(WebDriverCommand $command) {
     if ($command->getName() === DriverCommand::NEW_SESSION) {
       $this->service->start();
     }

lib/Remote/WebDriverBrowserType.php

@@ -19,7 +19,6 @@
  * All the browsers supported by selenium
  */
 class WebDriverBrowserType {
-
   const FIREFOX = "firefox";
   const FIREFOX_2 = "firefox2";
   const FIREFOX_3 = "firefox3";
@@ -42,4 +41,8 @@ class WebDriverBrowserType {
   const IPHONE = "iphone";
   const IPAD = "iPad";
   const PHANTOMJS = "phantomjs";
+
+  private function __construct()
+  {
+  }
 }

lib/Remote/WebDriverCapabilityType.php

@@ -16,11 +16,9 @@
 namespace Facebook\WebDriver\Remote;
 
 /**
- * WebDriverCapabilityType contains all constants defined in the WebDriver
- * Wire Protocol.
+ * WebDriverCapabilityType contains all constants defined in the WebDriver Wire Protocol.
  */
 class WebDriverCapabilityType {
-
   const BROWSER_NAME = 'browserName';
   const VERSION = 'version';
   const PLATFORM = 'platform';
@@ -37,4 +35,8 @@ class WebDriverCapabilityType {
   const ACCEPT_SSL_CERTS = 'acceptSslCerts';
   const NATIVE_EVENTS = 'nativeEvents';
   const PROXY = 'proxy';
+
+  private function __construct()
+  {
+  }
 }

lib/Remote/WebDriverCommand.php

@@ -16,25 +16,41 @@
 namespace Facebook\WebDriver\Remote;
 
 class WebDriverCommand {
-
+  /** @var string */
   private $sessionID;
+  /** @var string */
   private $name;
+  /** @var array */
   private $parameters;
 
+  /**
+   * @param string $session_id
+   * @param string $name Constant from DriverCommand
+   * @param array $parameters Array of
+   */
   public function __construct($session_id, $name, $parameters) {
     $this->sessionID = $session_id;
     $this->name = $name;
     $this->parameters = $parameters;
   }
 
+  /**
+   * @return string
+   */
   public function getName() {
     return $this->name;
   }
 
+  /**
+   * @return string
+   */
   public function getSessionID() {
     return $this->sessionID;
   }
 
+  /**
+   * @return array
+   */
   public function getParameters() {
     return $this->parameters;
   }

lib/WebDriverPlatform.php

@@ -19,7 +19,6 @@
  * The platforms supported by WebDriver.
  */
 class WebDriverPlatform {
-
   const ANDROID = 'ANDROID';
   const ANY = 'ANY';
   const LINUX = 'LINUX';
@@ -28,4 +27,8 @@ class WebDriverPlatform {
   const VISTA = 'VISTA';
   const WINDOWS = 'WINDOWS';
   const XP = 'XP';
+
+  private function __construct()
+  {
+  }
 }