added license docblock

Author: davert

lib/WebDriverKeys.php

@@ -84,7 +84,7 @@ class WebDriverKeys {
   const COMMAND         = "\xEE\x80\xBD"; // ALIAS
   const ZENKAKU_HANKAKU = "\xEE\x80\xC0";
 
-  static function encode($keys)
+  static function encode(array $keys)
   {
     $encoded = array();
     foreach ($keys as $key)

lib/__init__.php

@@ -25,6 +25,7 @@
 require_once('WebDriverKeys.php');
 require_once('WebDriverNavigation.php');
 require_once('WebDriverMouse.php');
+require_once('WebDriverKeyboard.php');
 require_once('WebDriverOptions.php');
 require_once('WebDriverPoint.php');
 require_once('WebDriverSelect.php');

lib/remote/RemoteKeyboard.php

@@ -1,6 +1,21 @@
 <?php
+// Copyright 2004-present Facebook. All Rights Reserved.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
 
-
+/**
+ * Execute keyboard commands for RemoteWebDriver.
+ */
 class RemoteKeyboard implements WebDriverKeyboard {
 
   private $executor;
@@ -18,7 +33,7 @@ public function __construct($executor) {
    * @return $this
    */
   public function sendKeys($keys) {
-    $this->sendKeysToActiveElement(WebDriverKeys::encode($keys));
+    $this->sendKeysToActiveElement(WebDriverKeys::encode(func_get_args()));
     return $this;
   }
 
@@ -31,8 +46,7 @@ public function sendKeys($keys) {
    */
   public function press($key)
   {
-    $this->ensureModifier($key);
-    $this->sendKeysToActiveElement(WebDriverKeys::encode(array($key)));
+    $this->sendKeysToActiveElement($this->getModifierKey($key));
     return $this;
   }
 
@@ -45,8 +59,7 @@ public function press($key)
    */
   public function release($key)
   {
-    $this->ensureModifier($key);
-    $this->sendKeysToActiveElement(WebDriverKeys::encode(array($key)));
+    $this->sendKeysToActiveElement($this->getModifierKey($key));
     return $this;
   }
 
@@ -58,9 +71,10 @@ private function sendKeysToActiveElement($value)
     $this->executor->execute('sendKeysToActiveElement', $params);   
   }    
 
-  private function ensureModifier($key)
+  private function getModifierKey($key)
   {
     if (!in_array($key, $this->modifiers))
       throw new InvalidArgumentException("$key is not a modifier key, expected one of ".implode(', ',$this->modifiers));
+    return array(constant('WebDriverKeys::'.strtoupper($key)));
   }
 }