Regard all modifier keys in SingleKeyAction

26814a1 added META, but COMMAND and LEFT_* were left out as
they were misinterpreted as aliases, which they aren't (they only
use the enum definition to get the same char code of the keys).

This time added a test case to avoid similar issues in the future.

Fixes
https://code.google.com/p/selenium/issues/detail?id=4843

Signed-off-by: Andreas Tolfsen <ato@mozilla.com>

java/client/src/org/openqa/selenium/interactions/internal/SingleKeyAction.java

@@ -27,7 +27,9 @@
  */
 public abstract class SingleKeyAction extends KeysRelatedAction {
   protected final Keys key;
-  private static final Keys[] MODIFIER_KEYS = {Keys.SHIFT, Keys.CONTROL, Keys.ALT, Keys.META};
+  private static final Keys[] MODIFIER_KEYS = {Keys.SHIFT, Keys.CONTROL, Keys.ALT, Keys.META,
+                                               Keys.COMMAND, Keys.LEFT_ALT, Keys.LEFT_CONTROL,
+                                               Keys.LEFT_SHIFT};
 
   protected SingleKeyAction(Keyboard keyboard, Mouse mouse, Keys key) {
     this(keyboard, mouse, null, key);

java/client/test/org/openqa/selenium/interactions/IndividualKeyboardActionsTest.java

@@ -135,4 +135,16 @@ public void keyDownActionFailsOnNonModifier() {
       assertTrue(e.getMessage().contains("modifier keys"));
     }
   }
+
+  @Test
+  public void testAllModifierKeysRegardedAsSuch() {
+    new KeyDownAction(mockKeyboard, mockMouse, stubLocatable, Keys.SHIFT);
+    new KeyDownAction(mockKeyboard, mockMouse, stubLocatable, Keys.LEFT_SHIFT);
+    new KeyDownAction(mockKeyboard, mockMouse, stubLocatable, Keys.CONTROL);
+    new KeyDownAction(mockKeyboard, mockMouse, stubLocatable, Keys.LEFT_CONTROL);
+    new KeyDownAction(mockKeyboard, mockMouse, stubLocatable, Keys.ALT);
+    new KeyDownAction(mockKeyboard, mockMouse, stubLocatable, Keys.LEFT_ALT);
+    new KeyDownAction(mockKeyboard, mockMouse, stubLocatable, Keys.META);
+    new KeyDownAction(mockKeyboard, mockMouse, stubLocatable, Keys.COMMAND);
+  }
 }