Linux accelerators cleanup

chrome/browser/renderer_host/render_widget_host_view_gtk.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 "chrome/browser/renderer_host/render_widget_host_view_gtk.h"
 
 #include <gtk/gtk.h>
 #include <gdk/gdk.h>
 #include <cairo/cairo.h>
 
@@ skipping 60 matching lines @@
                               RenderWidgetHostViewGtk* host_view) {
     const gfx::Rect damage_rect(expose->area);
     host_view->Paint(damage_rect);
     return FALSE;
   }
 
   static gboolean KeyPressReleaseEvent(GtkWidget* widget, GdkEventKey* event,
                                        RenderWidgetHostViewGtk* host_view) {
     NativeWebKeyboardEvent wke(event);
     host_view->GetRenderWidgetHost()->ForwardKeyboardEvent(wke);
-    return FALSE;
+    // We return TRUE because we did handle the event. If it turns out webkit
+    // can't handle the event, we'll deal with that later.

[Dean McNamee, 2009/03/16 19:19:02]

we deal with that in XXX.  This makes it sound like it's an open bug "yeah,
we'll fix that later"

+    return TRUE;
   }
 
   static gboolean FocusIn(GtkWidget* widget, GdkEventFocus* focus,
                           RenderWidgetHostViewGtk* host_view) {
     host_view->GetRenderWidgetHost()->Focus();
     return FALSE;
   }
 
   static gboolean FocusOut(GtkWidget* widget, GdkEventFocus* focus,
                            RenderWidgetHostViewGtk* host_view) {
@@ skipping 200 matching lines @@
     // Destroy()ed yet and it receives paint messages...
     GdkWindow* window = view_->window;
     if (window) {
       backing_store->ShowRect(
           damage_rect, x11_util::GetX11WindowFromGtkWidget(view_));
     }
   } else {
     NOTIMPLEMENTED();
   }
 }