diff options
Diffstat (limited to 'chrome/browser/extensions/extension_menu_manager.h')
-rw-r--r-- | chrome/browser/extensions/extension_menu_manager.h | 40 |
1 files changed, 4 insertions, 36 deletions
diff --git a/chrome/browser/extensions/extension_menu_manager.h b/chrome/browser/extensions/extension_menu_manager.h index c3f82c5..e85f5d5 100644 --- a/chrome/browser/extensions/extension_menu_manager.h +++ b/chrome/browser/extensions/extension_menu_manager.h @@ -14,14 +14,11 @@ #include "base/linked_ptr.h" #include "base/stl_util-inl.h" #include "base/string16.h" -#include "chrome/browser/extensions/image_loading_tracker.h" #include "chrome/common/notification_observer.h" #include "chrome/common/notification_registrar.h" -#include "third_party/skia/include/core/SkBitmap.h" struct ContextMenuParams; -class Extension; class ExtensionMessageService; class Profile; class TabContents; @@ -170,9 +167,7 @@ class ExtensionMenuItem { }; // This class keeps track of menu items added by extensions. -class ExtensionMenuManager - : public ImageLoadingTracker::Observer, - public NotificationObserver { +class ExtensionMenuManager : public NotificationObserver { public: ExtensionMenuManager(); virtual ~ExtensionMenuManager(); @@ -187,11 +182,9 @@ class ExtensionMenuManager // put them into a submenu if there are more than 1. const ExtensionMenuItem::List* MenuItems(const std::string& extension_id); - // Adds a top-level menu item for an extension, requiring the |extension| - // pointer so it can load the icon for the extension. Takes ownership of - // |item|. Returns the id assigned to the item. Has the side-effect of - // incrementing the next_item_id_ member. - int AddContextItem(Extension* extension, ExtensionMenuItem* item); + // Takes ownership of |item|. Returns the id assigned to the item. Has the + // side-effect of incrementing the next_item_id_ member. + int AddContextItem(ExtensionMenuItem* item); // Add an item as a child of another item which has been previously added, and // takes ownership of |item|. Returns the id assigned to the item, or 0 on @@ -224,15 +217,6 @@ class ExtensionMenuManager virtual void Observe(NotificationType type, const NotificationSource& source, const NotificationDetails& details); - // This returns a bitmap of width/height kFavIconSize, loaded either from an - // entry specified in the extension's 'icon' section of the manifest, or a - // default extension icon. - const SkBitmap& GetIconForExtension(const std::string& extension_id); - - // Implements the ImageLoadingTracker::Observer interface. - virtual void OnImageLoaded(SkBitmap* image, ExtensionResource resource, - int index); - private: // This is a helper function which takes care of de-selecting any other radio // items in the same group (i.e. that are adjacent in the list). @@ -241,13 +225,6 @@ class ExtensionMenuManager // Returns true if item is a descendant of an item with id |ancestor_id|. bool DescendantOf(ExtensionMenuItem* item, int ancestor_id); - // Makes sure we've done one-time initialization of the default extension icon - // default_icon_. - void EnsureDefaultIcon(); - - // Helper function to return a copy of |src| scaled to kFavIconSize. - SkBitmap ScaleToFavIconSize(const SkBitmap& src); - // We keep items organized by mapping an extension id to a list of items. typedef std::map<std::string, ExtensionMenuItem::List> MenuItemMap; MenuItemMap context_items_; @@ -262,15 +239,6 @@ class ExtensionMenuManager NotificationRegistrar registrar_; - // Used for loading extension icons. - ImageLoadingTracker image_tracker_; - - // Maps extension id to an SkBitmap with the icon for that extension. - std::map<std::string, SkBitmap> extension_icons_; - - // The default icon we'll use if an extension doesn't have one. - SkBitmap default_icon_; - DISALLOW_COPY_AND_ASSIGN(ExtensionMenuManager); }; |