summaryrefslogtreecommitdiffstats
path: root/chrome/browser/ui/toolbar
diff options
context:
space:
mode:
authoratwilson@chromium.org <atwilson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-02-01 18:06:19 +0000
committeratwilson@chromium.org <atwilson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-02-01 18:06:19 +0000
commitf94d7fee29a2fc40390bb00cfaca6e2f77165b2f (patch)
tree88c2e9200f2752b2f848691dceb91a9b8989ff86 /chrome/browser/ui/toolbar
parentd0a7adaf0b12a937d4de035930a345089d73f7ca (diff)
downloadchromium_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.cc21
-rw-r--r--chrome/browser/ui/toolbar/wrench_menu_model.h1
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;