Implement accelerators in the gtk port.

webkit/glue/webinputevent.h

 // 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.
 
 #ifndef WEBKIT_GLUE_WEBINPUTEVENT_H_
 #define WEBKIT_GLUE_WEBINPUTEVENT_H_
 
 #include "base/basictypes.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #elif defined(OS_MACOSX)
 #include <vector>
 #ifdef __OBJC__
 @class NSEvent;
 @class NSView;
 #else
 class NSEvent;
 class NSView;
 #endif  // __OBJC__
 #elif defined(OS_LINUX)
+#include <glib.h>
 typedef struct _GdkEventButton GdkEventButton;
 typedef struct _GdkEventMotion GdkEventMotion;
 typedef struct _GdkEventScroll GdkEventScroll;
 typedef struct _GdkEventKey GdkEventKey;
 #endif
 
 // The classes defined in this file are intended to be used with WebView's
 // HandleInputEvent method.  These event types are cross-platform; however,
 // there are platform-specific constructors that accept native UI events.
 //
@@ skipping 109 matching lines @@
   std::vector<unsigned short> key_identifier;
 #elif defined(OS_WIN)
   bool system_key;  // Set if we receive a SYSKEYDOWN/WM_SYSKEYUP message.
   MSG actual_message; // Set to the current keyboard message.
 #elif defined(OS_LINUX)
   // The unicode character, if available, corresponding to this key event.
   // TODO(evanm): temporary hack for test_shell.  Ideally we'd either manage
   // to stuff everything into key_code, or make this field shared by all
   // implementations, but this will have to do for now.
   wchar_t text;
+
+  // Stores the GDK keyval and modifier state of the GdkEventKey; this is used
+  // to transport these values across IPC so they can be used in the front end
+  // code.
+  guint gdk_keyval;
+  guint gdk_modifier;

[Avi (use Gerrit), 2009/02/25 22:38:40]

How is this different from the existing modifier info? Can you please reuse
fields?

[darin (slow to review), 2009/02/26 04:16:55]

yeah, i don't understand this either.  PlatformKeyboardEvent has no equivalent
fields, so this seems plainly wrong.

 #endif
 
   WebKeyboardEvent() 
       : key_code(0)
 #if defined(OS_WIN)
         , system_key(false) {
     memset(&actual_message, 0, sizeof(actual_message));
   }
 #else
   {}
 #endif
 
 #if defined(OS_WIN)
   WebKeyboardEvent(HWND hwnd, UINT message, WPARAM wparam, LPARAM lparam);
 #elif defined(OS_MACOSX)
   WebKeyboardEvent(NSEvent *event);
 #elif defined(OS_LINUX)
   explicit WebKeyboardEvent(const GdkEventKey* event);
 #endif
 };
 
 
 #endif  // WEBKIT_GLUE_WEBINPUTEVENT_H_