Creat a KeyboardEventManager class

third_party/WebKit/Source/platform/PlatformKeyboardEvent.cpp

 /*
  * Copyright (C) 2006, 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2008, 2009 Google Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 53 matching lines @@
             // happen to be in this range, e.g. the Apple logo (Option+Shift+K).
             // 0xF7FF is an arbitrary cut-off.
             m_text = String();
             m_unmodifiedText = String();
         }
 #endif
     }
 #endif
 }
 
+PlatformEvent::Modifiers PlatformKeyboardEvent::accessKeyModifiers()
+{
+#if OS(MACOSX)

[Rick Byers, 2016/06/08 17:58:16]

normally we prefer to avoid #ifdefs in blink for platform-specific behavior like
this, and instead use a setting that the embedder can set as appropriate to the
platform (I believe this is how we handle the contextmenu difference).  This
allows us to tests both behaviors on all OSes and have consistent test output.

But I see you're just moving this existing code from EventHandler, so cleaning
it up is not required.  Can you add a TODO though saying to move to a setting?

[Navid Zolghadr, 2016/06/08 18:55:08]

Done.

+    return static_cast<PlatformEvent::Modifiers>(PlatformEvent::CtrlKey | PlatformEvent::AltKey);
+#else
+    return PlatformEvent::AltKey;
+#endif
+}
+
 bool PlatformKeyboardEvent::currentCapsLockState()
 {
     switch (s_overrideCapsLockState) {
     case OverrideCapsLockState::Default:
 #if OS(WIN)
             // FIXME: Does this even work inside the sandbox?
             return GetKeyState(VK_CAPITAL) & 1;
 #elif OS(MACOSX)
             return GetCurrentKeyModifiers() & alphaLock;
 #else
@@ skipping 29 matching lines @@
     if (currentModifiers & ::cmdKey)
         modifiers |= MetaKey;
 #else
     // TODO(crbug.com/538289): Implement on other platforms.
     return static_cast<Modifiers>(0);
 #endif
     return static_cast<Modifiers>(modifiers);
 }
 
 } // namespace blink