DevTools: Forward whitelisted unhandled shortcuts from inspected page into DevTools frontend

chrome/browser/ui/browser.cc

 // Copyright 2012 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/ui/browser.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shellapi.h>
 #endif  // defined(OS_WIN)
@@ skipping 1131 matching lines @@
   // TODO(koz): Write a test for this http://crbug.com/100441.
   if (event.windowsKeyCode == 27 &&
       fullscreen_controller_->HandleUserPressedEscape()) {
     return true;
   }
   return window()->PreHandleKeyboardEvent(event, is_keyboard_shortcut);
 }
 
 void Browser::HandleKeyboardEvent(content::WebContents* source,
                                   const NativeWebKeyboardEvent& event) {
-  window()->HandleKeyboardEvent(event);
+  DevToolsWindow* dev_tools_window =
+      DevToolsWindow::GetInstanceForInspectedRenderViewHost(

[pfeldman, 2014/04/08 12:01:20]

Lets introduce GetInstanceForInspectedWebContents

[apavlov, 2014/04/08 13:28:12]

Done.

+          source->GetRenderViewHost());
+  bool handled = false;
+  if (dev_tools_window)
+    handled = dev_tools_window->ForwardKeyboardEvent(event);
+
+  if (!handled)
+    window()->HandleKeyboardEvent(event);
 }
 
 bool Browser::TabsNeedBeforeUnloadFired() {
   if (IsFastTabUnloadEnabled())
     return fast_unload_controller_->TabsNeedBeforeUnloadFired();
   return unload_controller_->TabsNeedBeforeUnloadFired();
 }
 
 void Browser::OverscrollUpdate(int delta_y) {
   window_->OverscrollUpdate(delta_y);
@@ skipping 1200 matching lines @@
   if (contents && !allow_js_access) {
     contents->web_contents()->GetController().LoadURL(
         target_url,
         content::Referrer(),
         content::PAGE_TRANSITION_LINK,
         std::string());  // No extra headers.
   }
 
   return contents != NULL;
 }