Fix rotation shortcut keys in PDF

pdf/instance.cc

Index: pdf/instance.cc
diff --git a/pdf/instance.cc b/pdf/instance.cc
index 4bd2f787dfec69d80534de5f9686dc5e619f32b9..10f2fafb6dfd7af456ef2a69c2a09da6a4cf1701 100644
--- a/pdf/instance.cc
+++ b/pdf/instance.cc
@@ -596,9 +596,6 @@ bool Instance::HandleInputEvent(const pp::InputEvent& event) {
         case 'A':
           engine_->SelectAll();
           return true;
-      }
-    } else if (modifier & PP_INPUTEVENT_MODIFIER_CONTROLKEY) {
-      switch (keyboard_event.GetKeyCode()) {

[raymes, 2014/11/03 05:23:56]

This doesn't look right to me. kDefaultKeyModifier is different on mac and
windows so I disagree that the if-statement is the same on both.

I can see why this might be a problem on non-mac platforms though - if the ctrl
key is held down we will enter the if-block but never the else- block. I think
the right solution here is to get rid of the "else" and just have two sequential
if-statements.

         case ui::VKEY_OEM_4:
           // Left bracket.
           engine_->RotateCounterclockwise();