Close detached DevTools window (rather than open a new instance) when it receives an F12 keypress

chrome/browser/ui/views/external_tab_container_win.cc

 // Copyright (c) 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/views/external_tab_container_win.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 1051 matching lines @@
       break;
     case IDC_ZOOM_NORMAL:
       host->Zoom(content::PAGE_ZOOM_RESET);
       break;
     case IDC_ZOOM_MINUS:
       host->Zoom(content::PAGE_ZOOM_OUT);
       break;
     case IDC_DEV_TOOLS:
       DevToolsWindow::ToggleDevToolsWindow(
           tab_contents_->web_contents()->GetRenderViewHost(),
-          DEVTOOLS_TOGGLE_ACTION_NONE);
+          false,
+          DEVTOOLS_TOGGLE_ACTION_SHOW);
       break;
     case IDC_DEV_TOOLS_CONSOLE:
       DevToolsWindow::ToggleDevToolsWindow(
           tab_contents_->web_contents()->GetRenderViewHost(),
+          false,
           DEVTOOLS_TOGGLE_ACTION_SHOW_CONSOLE);
       break;
     case IDC_DEV_TOOLS_INSPECT:
       DevToolsWindow::ToggleDevToolsWindow(
           tab_contents_->web_contents()->GetRenderViewHost(),
+          false,
           DEVTOOLS_TOGGLE_ACTION_INSPECT);
       break;
+    case IDC_DEV_TOOLS_TOGGLE:
+      DevToolsWindow::ToggleDevToolsWindow(
+          tab_contents_->web_contents()->GetRenderViewHost(),
+          false,
+          DEVTOOLS_TOGGLE_ACTION_TOGGLE);
+      break;
     default:
       NOTREACHED() << "Unsupported accelerator: " << command_id;
       return false;
   }
   return true;
 }
 
 bool ExternalTabContainerWin::CanHandleAccelerators() const {
   return true;
 }
@@ skipping 168 matching lines @@
   if (params.disposition == CURRENT_TAB) {
     DCHECK(route_all_top_level_navigations_);
     forward_params.disposition = NEW_FOREGROUND_TAB;
   }
   WebContents* new_contents =
       ExternalTabContainerWin::OpenURLFromTab(source, forward_params);
   // support only one navigation for a dummy tab before it is killed.
   ::DestroyWindow(GetNativeView());
   return new_contents;
 }