Don't perform the default action when keyCode is 229.

third_party/WebKit/Source/core/input/KeyboardEventManager.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "KeyboardEventManager.h"
 
 #include "core/dom/Element.h"
 #include "core/editing/Editor.h"
 #include "core/events/KeyboardEvent.h"
 #include "core/html/HTMLDialogElement.h"
@@ skipping 144 matching lines @@
     }
 }
 
 void KeyboardEventManager::defaultKeyboardEventHandler(
     KeyboardEvent* event, Node* possibleFocusedNode)
 {
     if (event->type() == EventTypeNames::keydown) {
         m_frame->editor().handleKeyboardEvent(event);
         if (event->defaultHandled())
             return;
+
+        // Do not perform the default action when inside a IME composition context.
+        // TODO(dtapuska): Replace this with isComposing support. crbug.com/625686
+        if (event->keyCode() == 229)

[bokan, 2016/07/04 22:52:06]

Does this keyCode have a name of some kind that we could replace 229 with a more
friendly constant?

+            return;
         if (event->key() == "Tab") {
             defaultTabEventHandler(event);
         } else if (event->key() == "Backspace") {
             defaultBackspaceEventHandler(event);
         } else if (event->key() == "Escape") {
             defaultEscapeEventHandler(event);
         } else {
             WebFocusType type = focusDirectionForKey(event->key());
             if (type != WebFocusTypeNone)
                 defaultArrowEventHandler(type, event);
@@ skipping 104 matching lines @@
     if (HTMLDialogElement* dialog = m_frame->document()->activeModalDialog())
         dialog->dispatchEvent(Event::createCancelable(EventTypeNames::cancel));
 }
 
 DEFINE_TRACE(KeyboardEventManager)
 {
     visitor->trace(m_frame);
 }
 
 } // namespace blink