Refactor the keyboard events handling code related to RenderViewHostDelegate:...

chrome/browser/views/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/views/tab_contents/tab_contents_view_gtk.h"
 
 #include <gdk/gdk.h>
 #include <gtk/gtk.h>
 
 #include "app/gfx/canvas_paint.h"
@@ skipping 296 matching lines @@
   tab_contents()->delegate()->TabContentsFocused(tab_contents());
 }
 
 void TabContentsViewGtk::TakeFocus(bool reverse) {
   // This is called when we the renderer asks us to take focus back (i.e., it
   // has iterated past the last focusable element on the page).
   gtk_widget_child_focus(GTK_WIDGET(GetTopLevelNativeWindow()),
       reverse ? GTK_DIR_TAB_BACKWARD : GTK_DIR_TAB_FORWARD);
 }
 
-bool TabContentsViewGtk::HandleKeyboardEvent(
-    const NativeWebKeyboardEvent& event) {
-  // The renderer returned a keyboard event it did not process. This may be
-  // a keyboard shortcut that we have to process.
-  if (event.type != WebInputEvent::RawKeyDown)
-    return false;
-
-  views::FocusManager* focus_manager =
-      views::FocusManager::GetFocusManagerForNativeView(GetNativeView());
-  // We may not have a focus_manager at this point (if the tab has been switched
-  // by the time this message returned).
-  if (!focus_manager)
-    return false;
-
-  bool shift_pressed = (event.modifiers & WebInputEvent::ShiftKey) ==
-                       WebInputEvent::ShiftKey;
-  bool ctrl_pressed = (event.modifiers & WebInputEvent::ControlKey) ==
-                      WebInputEvent::ControlKey;
-  bool alt_pressed = (event.modifiers & WebInputEvent::AltKey) ==
-                     WebInputEvent::AltKey;
-
-  return focus_manager->ProcessAccelerator(
-      views::Accelerator(static_cast<base::KeyboardCode>(event.windowsKeyCode),
-                         shift_pressed, ctrl_pressed, alt_pressed));
-  // DANGER: |this| could be deleted now!
-
-  // Note that we do not handle Gtk mnemonics/accelerators or binding set here
-  // (as it is done in BrowserWindowGtk::HandleKeyboardEvent), as we override
-  // Gtk behavior completely.
-}
-
 void TabContentsViewGtk::ShowContextMenu(const ContextMenuParams& params) {
   // Allow delegates to handle the context menu operation first.
   if (tab_contents()->delegate()->HandleContextMenu(params))
     return;
 
   context_menu_.reset(new RenderViewContextMenuWin(tab_contents(), params));
   context_menu_->Init();
 
   gfx::Point screen_point(params.x, params.y);
   views::View::ConvertPointToScreen(GetRootView(), &screen_point);
@@ skipping 80 matching lines @@
 gboolean TabContentsViewGtk::OnMouseMove(GtkWidget* widget,
                                          GdkEventMotion* event) {
   if (sad_tab_ != NULL)
     WidgetGtk::OnMotionNotify(widget, event);
   else if (tab_contents()->delegate())
     tab_contents()->delegate()->ContentsMouseEvent(
         tab_contents(), views::Screen::GetCursorScreenPoint(), true);
   return FALSE;
 }