Don't send tab switching/killing/creating keyboard accelerators to pages. Th...

chrome/browser/renderer_host/render_view_host.cc

Index: chrome/browser/renderer_host/render_view_host.cc
===================================================================
--- chrome/browser/renderer_host/render_view_host.cc	(revision 27783)
+++ chrome/browser/renderer_host/render_view_host.cc	(working copy)
@@ -1420,19 +1420,18 @@
   UserMetrics::RecordComputedAction(action.c_str(), process()->profile());
 }
 
+bool RenderViewHost::ShouldSendToRenderer(const NativeWebKeyboardEvent& event) {
+  RenderViewHostDelegate::View* view = delegate_->GetViewDelegate();
+  if (!view)
+    return true;
+  return !view->IsReservedAccelerator(event);
+}
+
 void RenderViewHost::UnhandledKeyboardEvent(
     const NativeWebKeyboardEvent& event) {
   RenderViewHostDelegate::View* view = delegate_->GetViewDelegate();
-  if (view) {
-    // TODO(brettw) why do we have to filter these types of events here. Can't
-    // the renderer just send us the ones we care abount, or maybe the view
-    // should be able to decide which ones it wants or not?
-    if ((event.type == WebInputEvent::RawKeyDown) ||
-        (event.type == WebInputEvent::KeyDown) ||
-        (event.type == WebInputEvent::Char)) {
-      view->HandleKeyboardEvent(event);
-    }
-  }
+  if (view)
+    view->HandleKeyboardEvent(event);
 }
 
 void RenderViewHost::OnUserGesture() {