Fixing WebKeyboardEvent.

webkit/glue/webinputevent_linux.cc

 // Copyright (c) 2006-2008 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 "config.h"
 
 #include "webkit/glue/webinputevent.h"
 
 #include "KeyboardCodes.h"
 #include "KeyCodeConversion.h"
 
-#include "webkit/glue/event_conversion.h"
-
-// This header is out of alphabetical order, but event_conversion.h pulls
-// in more webkit headers that redefine LOG so I need to undef afterwards.
-#undef LOG
 #include "base/logging.h"
+#include "base/string_util.h"
+#include "webkit/glue/webinputevent_utils.h"
 
 #include <gdk/gdk.h>
 #include <gdk/gdkkeysyms.h>
 #include <gtk/gtkversion.h>
 
 namespace {
 
 double GdkEventTimeToWebEventTime(guint32 time) {
   // Convert from time in ms to time in sec.
   return time / 1000.0;
@@ skipping 31 matching lines @@
       break;
     case GDK_2BUTTON_PRESS:
     case GDK_3BUTTON_PRESS:
       type = MOUSE_DOUBLE_CLICK;
       break;
     case GDK_BUTTON_RELEASE:
       type = MOUSE_UP;
       break;
 
     default:
-      ASSERT_NOT_REACHED();
+      NOTREACHED();
   };
 
   button = BUTTON_NONE;
   if (event->button == 1) {
     button = BUTTON_LEFT;
   } else if (event->button == 2) {
     button = BUTTON_MIDDLE;
   } else if (event->button == 3) {
     button = BUTTON_RIGHT;
   }
 }
 
 WebMouseEvent::WebMouseEvent(const GdkEventMotion* event) {
   timestamp_sec = GdkEventTimeToWebEventTime(event->time);
   modifiers = GdkStateToWebEventModifiers(event->state);
   x = static_cast<int>(event->x);
   y = static_cast<int>(event->y);
   global_x = static_cast<int>(event->x_root);
   global_y = static_cast<int>(event->y_root);
 
   switch (event->type) {
     case GDK_MOTION_NOTIFY:
       type = MOUSE_MOVE;
       break;
     default:
-      ASSERT_NOT_REACHED();
+      NOTREACHED();
   }
 
   button = BUTTON_NONE;
   if (event->state & GDK_BUTTON1_MASK) {
     button = BUTTON_LEFT;
   } else if (event->state & GDK_BUTTON2_MASK) {
     button = BUTTON_MIDDLE;
   } else if (event->state & GDK_BUTTON3_MASK) {
     button = BUTTON_RIGHT;
   }
@@ skipping 27 matching lines @@
       break;
     case GDK_SCROLL_RIGHT:
       delta_x = kWheelDelta;
       break;
     default:
       break;
   }
 }
 
 WebKeyboardEvent::WebKeyboardEvent(const GdkEventKey* event) {
+  system_key = false;
   modifiers = GdkStateToWebEventModifiers(event->state);
 
-  // GDK only exposes key press and release events.  By contrast,
-  // WebKeyboardEvent matches Windows and wants key down/up events along with a
-  // separate CHAR event.
-  // We require the caller to simulate the CHAR event manually.  See
-  // test_shell's webwidget_host for an example.
   switch (event->type) {
     case GDK_KEY_RELEASE:
       type = KEY_UP;
       break;
     case GDK_KEY_PRESS:
       type = KEY_DOWN;
       break;
     default:
       NOTREACHED();
       break;
   }
 
   // The key code tells us which physical key was pressed (for example, the
   // A key went down or up).  It does not determine whether A should be lower
   // or upper case.  This is what text does, which should be the keyval.
-  key_code = WebCore::windowsKeyCodeForKeyEvent(event->keyval);
+  windows_key_code = WebCore::windowsKeyCodeForKeyEvent(event->keyval);
+  native_key_code = event->hardware_keycode;
+
+  memset(&text, 0, sizeof(text));
+  memset(&unmodified_text, 0, sizeof(unmodified_text));
+  memset(&key_identifier, 0, sizeof(key_identifier));
 
   switch (event->keyval) {
     // We need to treat the enter key as a key press of character \r.  This
     // is apparently just how webkit handles it and what it expects.
     case GDK_ISO_Enter:
     case GDK_KP_Enter:
     case GDK_Return:
-      text = '\r';
+      unmodified_text[0] = text[0] = static_cast<char16>('\r');
       break;
     default:
       // This should set text to 0 when it's not a real character.
-      text = gdk_keyval_to_unicode(event->keyval);
+      // TODO(avi): fix for non BMP chars
+      unmodified_text[0] = text[0] =
+          static_cast<char16>(gdk_keyval_to_unicode(event->keyval));
       break;
   }
 
+  std::string key_identifier_str =
+      GetKeyIdentifierForWindowsKeyCode(windows_key_code);
+  base::strlcpy(key_identifier, key_identifier_str.c_str(),
+                kIdentifierLengthCap);
+
   // TODO(tc): Do we need to set IS_AUTO_REPEAT or IS_KEYPAD?
 }