diff options
author | apavlov@chromium.org <apavlov@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-16 11:58:48 +0000 |
---|---|---|
committer | apavlov@chromium.org <apavlov@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-16 11:58:48 +0000 |
commit | 2a8a98126b0115e0f3d137668c67b90a399f72bf (patch) | |
tree | ac1eedcdf0439b6c867cdce638d8a512d7cabaee /chrome/browser/browser.cc | |
parent | 8664dfb1f41cb9c51e50b3f7c4d78b24d29408d8 (diff) | |
download | chromium_src-2a8a98126b0115e0f3d137668c67b90a399f72bf.zip chromium_src-2a8a98126b0115e0f3d137668c67b90a399f72bf.tar.gz chromium_src-2a8a98126b0115e0f3d137668c67b90a399f72bf.tar.bz2 |
Add "Inspect Element" shortcut to browser tabs.
If the DevTools window is closed, it gets opened, and the Inspect Element tool is enabled. If the DevTools window is opened, the inspected tab remains focused with the Inspect Element tool enabled.
If the Inspect Element tool is enabled, it gets turned off.
The shortcut is Ctrl-Shift-C for Win/Linux, Cmd-Shift-C for Mac
BUG=29532
TEST=manual
Review URL: http://codereview.chromium.org/3011002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52651 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser.cc')
-rw-r--r-- | chrome/browser/browser.cc | 35 |
1 files changed, 25 insertions, 10 deletions
diff --git a/chrome/browser/browser.cc b/chrome/browser/browser.cc index c4a25df..dd8abe1 100644 --- a/chrome/browser/browser.cc +++ b/chrome/browser/browser.cc @@ -33,6 +33,7 @@ #include "chrome/browser/browser_url_handler.h" #include "chrome/browser/character_encoding.h" #include "chrome/browser/debugger/devtools_manager.h" +#include "chrome/browser/debugger/devtools_toggle_action.h" #include "chrome/browser/debugger/devtools_window.h" #include "chrome/browser/dock_info.h" #include "chrome/browser/dom_ui/filebrowse_ui.h" @@ -1669,15 +1670,21 @@ void Browser::OpenCreateShortcutsDialog() { #endif } -void Browser::ToggleDevToolsWindow(bool open_console) { - if (open_console) - UserMetrics::RecordAction(UserMetricsAction("DevTools_ToggleConsole"), - profile_); - else - UserMetrics::RecordAction(UserMetricsAction("DevTools_ToggleWindow"), - profile_); +void Browser::ToggleDevToolsWindow(DevToolsToggleAction action) { + std::string uma_string; + switch (action) { + case DEVTOOLS_TOGGLE_ACTION_SHOW_CONSOLE: + uma_string = "DevTools_ToggleConsole"; + break; + case DEVTOOLS_TOGGLE_ACTION_NONE: + case DEVTOOLS_TOGGLE_ACTION_INSPECT: + default: + uma_string = "DevTools_ToggleWindow"; + break; + } + UserMetrics::RecordAction(UserMetricsAction(uma_string.c_str()), profile_); DevToolsManager::GetInstance()->ToggleDevToolsWindow( - GetSelectedTabContents()->render_view_host(), open_console); + GetSelectedTabContents()->render_view_host(), action); } void Browser::OpenTaskManager() { @@ -2085,8 +2092,15 @@ void Browser::ExecuteCommandWithDisposition( // Show various bits of UI case IDC_OPEN_FILE: OpenFile(); break; case IDC_CREATE_SHORTCUTS: OpenCreateShortcutsDialog(); break; - case IDC_DEV_TOOLS: ToggleDevToolsWindow(false); break; - case IDC_DEV_TOOLS_CONSOLE: ToggleDevToolsWindow(true); break; + case IDC_DEV_TOOLS: ToggleDevToolsWindow( + DEVTOOLS_TOGGLE_ACTION_NONE); + break; + case IDC_DEV_TOOLS_CONSOLE: ToggleDevToolsWindow( + DEVTOOLS_TOGGLE_ACTION_SHOW_CONSOLE); + break; + case IDC_DEV_TOOLS_INSPECT: ToggleDevToolsWindow( + DEVTOOLS_TOGGLE_ACTION_INSPECT); + break; case IDC_TASK_MANAGER: OpenTaskManager(); break; case IDC_REPORT_BUG: OpenBugReportDialog(); break; @@ -3164,6 +3178,7 @@ void Browser::InitCommandState() { command_updater_.UpdateCommandEnabled(IDC_CREATE_SHORTCUTS, false); command_updater_.UpdateCommandEnabled(IDC_DEV_TOOLS, true); command_updater_.UpdateCommandEnabled(IDC_DEV_TOOLS_CONSOLE, true); + command_updater_.UpdateCommandEnabled(IDC_DEV_TOOLS_INSPECT, true); command_updater_.UpdateCommandEnabled(IDC_TASK_MANAGER, true); command_updater_.UpdateCommandEnabled(IDC_SHOW_HISTORY, true); command_updater_.UpdateCommandEnabled(IDC_SHOW_BOOKMARK_MANAGER, true); |