diff options
author | atwilson@chromium.org <atwilson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-01 18:06:19 +0000 |
---|---|---|
committer | atwilson@chromium.org <atwilson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-01 18:06:19 +0000 |
commit | f94d7fee29a2fc40390bb00cfaca6e2f77165b2f (patch) | |
tree | 88c2e9200f2752b2f848691dceb91a9b8989ff86 /chrome/browser/ui/toolbar | |
parent | d0a7adaf0b12a937d4de035930a345089d73f7ca (diff) | |
download | chromium_src-f94d7fee29a2fc40390bb00cfaca6e2f77165b2f.zip chromium_src-f94d7fee29a2fc40390bb00cfaca6e2f77165b2f.tar.gz chromium_src-f94d7fee29a2fc40390bb00cfaca6e2f77165b2f.tar.bz2 |
Removed icon from View Background Pages menu item in wrench menu.
BUG=71489
TEST=Install extension with background page, pull up wrench menu, see no icon next to View Background Pages item.
Review URL: http://codereview.chromium.org/6347062
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73305 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/toolbar')
-rw-r--r-- | chrome/browser/ui/toolbar/wrench_menu_model.cc | 21 | ||||
-rw-r--r-- | chrome/browser/ui/toolbar/wrench_menu_model.h | 1 |
2 files changed, 0 insertions, 22 deletions
diff --git a/chrome/browser/ui/toolbar/wrench_menu_model.cc b/chrome/browser/ui/toolbar/wrench_menu_model.cc index 2028512..a1151df 100644 --- a/chrome/browser/ui/toolbar/wrench_menu_model.cc +++ b/chrome/browser/ui/toolbar/wrench_menu_model.cc @@ -224,27 +224,6 @@ bool WrenchMenuModel::IsItemForCommandIdDynamic(int command_id) const { command_id == IDC_VIEW_BACKGROUND_PAGES; } -bool WrenchMenuModel::GetIconForCommandId(int command_id, - SkBitmap* bitmap) const { - switch (command_id) { - case IDC_VIEW_BACKGROUND_PAGES: { - int num_pages = BackgroundPageTracker::GetInstance()-> - GetUnacknowledgedBackgroundPageCount(); - if (num_pages > 0) { - *bitmap = *ResourceBundle::GetSharedInstance().GetBitmapNamed( - IDR_BACKGROUND_MENU); - return true; - } else { - // No icon. - return false; - } - } - default: - // No icon for other dynamic menu items. - return false; - } -} - string16 WrenchMenuModel::GetLabelForCommandId(int command_id) const { switch (command_id) { case IDC_SYNC_BOOKMARKS: diff --git a/chrome/browser/ui/toolbar/wrench_menu_model.h b/chrome/browser/ui/toolbar/wrench_menu_model.h index 8886de2..3b95383 100644 --- a/chrome/browser/ui/toolbar/wrench_menu_model.h +++ b/chrome/browser/ui/toolbar/wrench_menu_model.h @@ -84,7 +84,6 @@ class WrenchMenuModel : public ui::SimpleMenuModel, // Overridden for both ButtonMenuItemModel::Delegate and SimpleMenuModel: virtual bool IsItemForCommandIdDynamic(int command_id) const; virtual string16 GetLabelForCommandId(int command_id) const; - virtual bool GetIconForCommandId(int command_id, SkBitmap* icon) const; virtual void ExecuteCommand(int command_id); virtual bool IsCommandIdChecked(int command_id) const; virtual bool IsCommandIdEnabled(int command_id) const; |