Reverting key change (again); this breaks every keyboard layout test there is...

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:
-      NOTREACHED();
+      ASSERT_NOT_REACHED();
   };
 
   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:
-      NOTREACHED();
+      ASSERT_NOT_REACHED();
   }
 
   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 36 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.
-  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));
+  key_code = WebCore::windowsKeyCodeForKeyEvent(event->keyval);
 
   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:
-      unmodified_text[0] = text[0] = static_cast<char16>('\r');
+      text = '\r';
       break;
     default:
       // This should set text to 0 when it's not a real character.
-      // TODO(avi): fix for non BMP chars
-      unmodified_text[0] = text[0] =
-          static_cast<char16>(gdk_keyval_to_unicode(event->keyval));
+      text = 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?
 }