Switch from text insertion to key press and release events on the virtual k…

ui/keyboard/keyboard_util.cc

 // Copyright (c) 2013 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 "ui/keyboard/keyboard_util.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/strings/string16.h"
 #include "grit/keyboard_resources.h"
 #include "grit/keyboard_resources_map.h"
 #include "ui/aura/client/aura_constants.h"
 #include "ui/aura/root_window.h"
 #include "ui/base/ime/input_method.h"
 #include "ui/base/ime/text_input_client.h"
 #include "ui/keyboard/keyboard_switches.h"
 
+namespace {
+
+const char kKeyDown[] ="keydown";
+const char kKeyUp[] = "keyup";
+
+}  // namespace
+
 namespace keyboard {
 
 bool IsKeyboardEnabled() {
   return CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kEnableVirtualKeyboard);
 }
 
 bool InsertText(const base::string16& text, aura::RootWindow* root_window) {
   if (!root_window)
     return false;
 
-  // Handle Backspace and Enter specially: using TextInputClient::InsertText is
-  // very unreliable for these characters.
-  // TODO(bryeung): remove this code once virtual keyboards are able to send
-  // these events directly via the Input Injection API.
-  if (text.length() == 1) {
-    ui::KeyboardCode code = ui::VKEY_UNKNOWN;
-    if (text[0] == L'\n')
-      code = ui::VKEY_RETURN;
-    else if (text[0] == L'\b')
-      code = ui::VKEY_BACK;
-
-    if (code != ui::VKEY_UNKNOWN) {
-      ui::KeyEvent press_event(ui::ET_KEY_PRESSED, code, 0, 0);
-      root_window->AsRootWindowHostDelegate()->OnHostKeyEvent(&press_event);
-
-      ui::KeyEvent release_event(ui::ET_KEY_RELEASED, code, 0, 0);
-      root_window->AsRootWindowHostDelegate()->OnHostKeyEvent(&release_event);
-
-      return true;
-    }
-  }
-
   ui::InputMethod* input_method = root_window->GetProperty(
       aura::client::kRootWindowInputMethodKey);
   if (!input_method)
     return false;
 
   ui::TextInputClient* tic = input_method->GetTextInputClient();
   if (!tic || tic->GetTextInputType() == ui::TEXT_INPUT_TYPE_NONE)
     return false;
 
   tic->InsertText(text);
@@ skipping 33 matching lines @@
   // Then deal with the y movement.
   if (codey != ui::VKEY_UNKNOWN) {
     ui::KeyEvent press_event(ui::ET_KEY_PRESSED, codey, modifier_flags, 0);
     root_window->AsRootWindowHostDelegate()->OnHostKeyEvent(&press_event);
     ui::KeyEvent release_event(ui::ET_KEY_RELEASED, codey, modifier_flags, 0);
     root_window->AsRootWindowHostDelegate()->OnHostKeyEvent(&release_event);
   }
   return true;
 }
 
+bool SendKeyEvent(const std::string type,
+                  int key_value,
+                  int key_code,
+                  bool shift_modifier,
+                  aura::RootWindow* root_window) {
+  ui::EventType event_type = ui::ET_UNKNOWN;
+  if (type == kKeyDown)
+    event_type = ui::ET_KEY_PRESSED;
+  else if (type == kKeyUp)
+    event_type = ui::ET_KEY_RELEASED;
+  if (event_type == ui::ET_UNKNOWN)
+    return false;
+
+  int flags = ui::EF_NONE;
+  if (shift_modifier)
+    flags = ui::EF_SHIFT_DOWN;
+
+  ui::KeyboardCode code = static_cast<ui::KeyboardCode>(key_code);
+
+  ui::KeyEvent event(event_type, code, flags, false);
+  event.set_character(key_value);
+  event.set_unmodified_character(key_value);
+
+  if (code != ui::VKEY_UNKNOWN) {
+    root_window->AsRootWindowHostDelegate()->OnHostKeyEvent(&event);
+  } else if (event_type == ui::ET_KEY_RELEASED) {
+    // TODO(kevers): Fix key handling to support key_value when code is
+    // VKEY_UNKNOWN.
+    base::string16 text;
+    text.push_back(static_cast<char16>(key_value));
+    InsertText(text, root_window);
+  }
+  return true;
+}
+
 const GritResourceMap* GetKeyboardExtensionResources(size_t* size) {
   // This looks a lot like the contents of a resource map; however it is
   // necessary to have a custom path for the extension path, so the resource
   // map cannot be used directly.
   static const GritResourceMap kKeyboardResources[] = {
     {"keyboard/api_adapter.js", IDR_KEYBOARD_API_ADAPTER_JS},
     {"keyboard/constants.js", IDR_KEYBOARD_CONSTANTS_JS},
     {"keyboard/elements/kb-altkey.html", IDR_KEYBOARD_ELEMENTS_ALTKEY},
     {"keyboard/elements/kb-altkey-container.html",
         IDR_KEYBOARD_ELEMENTS_ALTKEY_CONTAINER},
@@ skipping 26 matching lines @@
     {"keyboard/main.css", IDR_KEYBOARD_MAIN_CSS},
     {"keyboard/polymer.min.js", IDR_KEYBOARD_POLYMER},
     {"keyboard/voice_input.js", IDR_KEYBOARD_VOICE_INPUT_JS},
   };
   static const size_t kKeyboardResourcesSize = arraysize(kKeyboardResources);
   *size = kKeyboardResourcesSize;
   return kKeyboardResources;
 }
 
 }  // namespace keyboard