Integrating GtkIMContext into the RenderWidgetHostViewGtk class (Take 2)....

chrome/browser/tab_contents/tab_contents_view_gtk.cc

 // Copyright (c) 2009 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 "chrome/browser/tab_contents/tab_contents_view_gtk.h"
 
 #include <gdk/gdk.h>
 #include <gtk/gtk.h>
 
 #include "base/string_util.h"
@@ skipping 290 matching lines @@
 void TabContentsViewGtk::HandleKeyboardEvent(
     const NativeWebKeyboardEvent& event) {
   // This may be an accelerator. Try to pass it on to our browser window
   // to handle.
   GtkWindow* window = GetTopLevelNativeWindow();
   if (!window) {
     NOTREACHED();
     return;
   }
 
+  // Filter out pseudo key events created by GtkIMContext signal handlers.
+  // Since GtkIMContext signal handlers don't use GdkEventKey objects, its
+  // |event.os_event| values are dummy values (or NULL.)
+  // We should filter out these pseudo key events to prevent unexpected
+  // behaviors caused by them.
+  if (event.type == WebKit::WebInputEvent::Char)
+    return;
+
   BrowserWindowGtk* browser_window =
       BrowserWindowGtk::GetBrowserWindowForNativeWindow(window);
   DCHECK(browser_window);
   browser_window->HandleAccelerator(event.os_event->keyval,
       static_cast<GdkModifierType>(event.os_event->state));
 }
 
 void TabContentsViewGtk::Observe(NotificationType type,
                                  const NotificationSource& source,
                                  const NotificationDetails& details) {
@@ skipping 105 matching lines @@
     gtk_container_child_set_property(GTK_CONTAINER(floating_container),
                                      widget, "x", &value);
 
     int child_y = std::max(half_view_height - (requisition.height / 2), 0);
     g_value_set_int(&value, child_y);
     gtk_container_child_set_property(GTK_CONTAINER(floating_container),
                                      widget, "y", &value);
     g_value_unset(&value);
   }
 }