Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2248)

Unified Diff: chrome/browser/ui/browser_command_controller.cc

Issue 10823233: Close detached DevTools window (rather than open a new instance) when it receives an F12 keypress (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Return NULL as the ToggleDevToolsWindow() result for DevTools windows Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/browser_command_controller.cc
diff --git a/chrome/browser/ui/browser_command_controller.cc b/chrome/browser/ui/browser_command_controller.cc
index c316c7e384067dd0a866f1245adcc4cdf07e5129..c0268ffa734d1e1fecdfbf95f4fbae226ce33d1f 100644
--- a/chrome/browser/ui/browser_command_controller.cc
+++ b/chrome/browser/ui/browser_command_controller.cc
@@ -485,7 +485,7 @@ void BrowserCommandController::ExecuteCommandWithDisposition(
CreateApplicationShortcuts(browser_);
break;
case IDC_DEV_TOOLS:
- ToggleDevToolsWindow(browser_, DEVTOOLS_TOGGLE_ACTION_NONE);
+ ToggleDevToolsWindow(browser_, DEVTOOLS_TOGGLE_ACTION_SHOW);
break;
case IDC_DEV_TOOLS_CONSOLE:
ToggleDevToolsWindow(browser_, DEVTOOLS_TOGGLE_ACTION_SHOW_CONSOLE);
@@ -493,6 +493,9 @@ void BrowserCommandController::ExecuteCommandWithDisposition(
case IDC_DEV_TOOLS_INSPECT:
ToggleDevToolsWindow(browser_, DEVTOOLS_TOGGLE_ACTION_INSPECT);
break;
+ case IDC_DEV_TOOLS_TOGGLE:
+ ToggleDevToolsWindow(browser_, DEVTOOLS_TOGGLE_ACTION_TOGGLE);
+ break;
case IDC_TASK_MANAGER:
OpenTaskManager(browser_, false);
break;
@@ -926,6 +929,8 @@ void BrowserCommandController::UpdateCommandsForDevTools() {
dev_tools_enabled);
command_updater_.UpdateCommandEnabled(IDC_DEV_TOOLS_INSPECT,
dev_tools_enabled);
+ command_updater_.UpdateCommandEnabled(IDC_DEV_TOOLS_TOGGLE,
+ dev_tools_enabled);
}
void BrowserCommandController::UpdateCommandsForBookmarkEditing() {

Powered by Google App Engine
This is Rietveld 408576698