Creat a KeyboardEventManager class

third_party/WebKit/Source/platform/PlatformKeyboardEvent.cpp

Index: third_party/WebKit/Source/platform/PlatformKeyboardEvent.cpp
diff --git a/third_party/WebKit/Source/platform/PlatformKeyboardEvent.cpp b/third_party/WebKit/Source/platform/PlatformKeyboardEvent.cpp
index f5b0d9084250359e0a8279b93cde73df8fd5a205..5da7650823428d28351733310879dc92ce959716 100644
--- a/third_party/WebKit/Source/platform/PlatformKeyboardEvent.cpp
+++ b/third_party/WebKit/Source/platform/PlatformKeyboardEvent.cpp
@@ -71,6 +71,15 @@ void PlatformKeyboardEvent::disambiguateKeyDownEvent(EventType type)
 #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) {