[Events] Propagate IME composition event to JS on Android.

Source/core/events/KeyboardEvent.cpp

 /**
  * Copyright (C) 2001 Peter Kelly (pmk@post.com)
  * Copyright (C) 2001 Tobias Anton (anton@stud.fbi.fh-darmstadt.de)
  * Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2003, 2005, 2006, 2007 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 10 matching lines @@
  */
 
 #include "config.h"
 #include "core/events/KeyboardEvent.h"
 
 #include "platform/PlatformKeyboardEvent.h"
 #include "platform/WindowsKeyboardCodes.h"
 
 namespace blink {
 
+namespace {
+
+#if OS(ANDROID)
+int kInputMethodEditorCompositionKeyCode = 229;

[bcwhite, 2015/01/12 21:02:44]

Is this constant available anywhere else already or can it be read from Android
SDK code?

[huangs, 2015/01/12 23:25:57]

Searched for 229 and 0xE5.  Something close is

https://code.google.com/p/chromium/codesearch#chromium/src/content/shell/rend...

which calls it VKEY_PROCESSKEY, but still defines it anyway.

Copying the style to reduce delta, and to facilitate future refactoring.

+#endif
+
+} // namespace
+
 static inline const AtomicString& eventTypeForKeyboardEventType(PlatformEvent::Type type)
 {
     switch (type) {
         case PlatformEvent::KeyUp:
             return EventTypeNames::keyup;
         case PlatformEvent::RawKeyDown:
             return EventTypeNames::keydown;
         case PlatformEvent::Char:
             return EventTypeNames::keypress;
         case PlatformEvent::KeyDown:
@@ skipping 115 matching lines @@
     return false;
 }
 
 int KeyboardEvent::keyCode() const
 {
     // IE: virtual key code for keyup/keydown, character code for keypress
     // Firefox: virtual key code for keyup/keydown, zero for keypress
     // We match IE.
     if (!m_keyEvent)
         return 0;
+
+#if OS(ANDROID)

[bcwhite, 2015/01/12 21:02:44]

// TODO: Check if this applies to other OS

[huangs, 2015/01/12 23:25:57]

Done (using FIXME instead).

+    // If the key event belongs to IME composition then propagate to JS.
+    if (m_keyEvent->nativeVirtualKeyCode() == kInputMethodEditorCompositionKeyCode)
+        return kInputMethodEditorCompositionKeyCode;
+#endif
+
     if (type() == EventTypeNames::keydown || type() == EventTypeNames::keyup)
         return windowsVirtualKeyCodeWithoutLocation(m_keyEvent->windowsVirtualKeyCode());
 
     return charCode();
 }
 
 int KeyboardEvent::charCode() const
 {
     // IE: not supported
     // Firefox: 0 for keydown/keyup events, character code for keypress
@@ skipping 37 matching lines @@
 {
 }
 
 bool KeyboardEventDispatchMediator::dispatchEvent(EventDispatcher& dispatcher) const
 {
     // Make sure not to return true if we already took default action while handling the event.
     return EventDispatchMediator::dispatchEvent(dispatcher) && !event().defaultHandled();
 }
 
 } // namespace blink