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

chrome/browser/browser_window.h

 // Copyright (c) 2006-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.
 
 #ifndef CHROME_BROWSER_BROWSER_WINDOW_H_
 #define CHROME_BROWSER_BROWSER_WINDOW_H_
 
 #include "app/gfx/native_widget_types.h"
 #include "chrome/browser/tab_contents/navigation_entry.h"
 
@@ skipping 249 matching lines @@
                             const GURL& url,
                             const NavigationEntry::SSLStatus& ssl,
                             bool show_history) = 0;
 
   // Shows the page menu (for accessibility).
   virtual void ShowPageMenu() = 0;
 
   // Shows the app menu (for accessibility).
   virtual void ShowAppMenu() = 0;
 
-  // Returns the id of the keyboard accelerator associated with the given
-  // keyboard event if one exists, otherwise -1.
-  virtual int GetCommandId(const NativeWebKeyboardEvent& event) = 0;
+  // Allows the BrowserWindow object to handle the specified keyboard event
+  // before sending it to the renderer.
+  // Returns true if the |event| was handled. Otherwise, if the |event| would
+  // be handled in HandleKeyboardEvent() method as a normal keyboard shortcut,
+  // |*is_keyboard_shortcut| should be set to true.
+  virtual bool PreHandleKeyboardEvent(const NativeWebKeyboardEvent& event,
+                                      bool* is_keyboard_shortcut) = 0;
+
+  // Allows the BrowserWindow object to handle the specified keyboard event,
+  // if the renderer did not process it.
+  virtual void HandleKeyboardEvent(const NativeWebKeyboardEvent& event) = 0;
 
   // Shows the create web app shortcut dialog box.
   virtual void ShowCreateShortcutsDialog(TabContents* tab_contents) = 0;
 
 #if defined(TOOLKIT_VIEWS)
   // Toggles compact navigation bar.
   virtual void ToggleCompactNavigationBar() = 0;
 #endif  // defined(TOOLKIT_VIEWS)
 
   // Construct a BrowserWindow implementation for the specified |browser|.
@@ skipping 32 matching lines @@
 
   // Returns the TabContentsContainer.
   virtual views::View* GetTabContentsContainerView() const = 0;
 
   // Returns the ToolbarView.
   virtual ToolbarView* GetToolbarView() const = 0;
 #endif
 };
 
 #endif  // CHROME_BROWSER_BROWSER_WINDOW_H_