diff options
| author | sadrul@chromium.org <sadrul@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-24 18:58:07 +0000 |
|---|---|---|
| committer | sadrul@chromium.org <sadrul@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-24 18:58:07 +0000 |
| commit | 23214631f9f132f7f08c8e13f2137b7334dd6aea (patch) | |
| tree | 69981d81034b03bbdd239355e005d8d54bc45af1 /chrome/browser/tabs/tab_strip_model.h | |
| parent | 93dbabaadefbfc5750e5b7284c1ebd005e006182 (diff) | |
| download | chromium_src-23214631f9f132f7f08c8e13f2137b7334dd6aea.zip chromium_src-23214631f9f132f7f08c8e13f2137b7334dd6aea.tar.gz chromium_src-23214631f9f132f7f08c8e13f2137b7334dd6aea.tar.bz2 | |
views: Show accelerators in tabstrip context menu.
As it happens, the command-id is a TabStripModel::ContextMenuCommand. So it
needs to be converted into a browser command first, before the accelerator is
looked up.
BUG=None
TEST=Right click on a tab. The context menu should have proper accelerators
displayed (e.g. Ctrl+T for the 'New Tab' option).
Review URL: http://codereview.chromium.org/5331005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67289 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/tabs/tab_strip_model.h')
| -rw-r--r-- | chrome/browser/tabs/tab_strip_model.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/chrome/browser/tabs/tab_strip_model.h b/chrome/browser/tabs/tab_strip_model.h index 8d385cb..2aae16e 100644 --- a/chrome/browser/tabs/tab_strip_model.h +++ b/chrome/browser/tabs/tab_strip_model.h @@ -411,6 +411,10 @@ class TabStripModel : public NotificationObserver { const NotificationSource& source, const NotificationDetails& details); + // Convert a ContextMenuCommand into a browser command. Returns true if a + // corresponding browser command exists, false otherwise. + static bool ContextMenuCommandToBrowserCommand(int cmd_id, int* browser_cmd); + private: // We cannot be constructed without a delegate. TabStripModel(); |