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

chrome/browser/renderer_host/render_view_host_delegate.h

Index: chrome/browser/renderer_host/render_view_host_delegate.h
===================================================================
--- chrome/browser/renderer_host/render_view_host_delegate.h	(revision 34219)
+++ chrome/browser/renderer_host/render_view_host_delegate.h	(working copy)
@@ -126,15 +126,18 @@
     // true, it means the focus was retrieved by doing a Shift-Tab.
     virtual void TakeFocus(bool reverse) = 0;
 
-    // Returns whether the event is a reserved keyboard shortcut that should not
-    // be sent to the renderer.
-    virtual bool IsReservedAccelerator(const NativeWebKeyboardEvent& event) = 0;
+    // Callback to give the browser a chance 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;
 
     // Callback to inform the browser that the renderer did not process the
     // specified events. This gives an opportunity to the browser to process the
     // event (used for keyboard shortcuts).
-    // Returns true if the event was handled.
-    virtual bool HandleKeyboardEvent(const NativeWebKeyboardEvent& event) = 0;
+    virtual void HandleKeyboardEvent(const NativeWebKeyboardEvent& event) = 0;
 
     // Notifications about mouse events in this view.  This is useful for
     // implementing global 'on hover' features external to the view.