Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(526)

Unified Diff: chrome/browser/ui/app_list/extension_app_item.h

Issue 2787693002: Remove more dead app list code. (Closed)
Patch Set: updated comment Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/app_list/extension_app_item.h
diff --git a/chrome/browser/ui/app_list/extension_app_item.h b/chrome/browser/ui/app_list/extension_app_item.h
index b65c88adffa33d397f5f1219fc5dacffbb29d6e3..18cadae2d47135b72442083b72395c291c243ce1 100644
--- a/chrome/browser/ui/app_list/extension_app_item.h
+++ b/chrome/browser/ui/app_list/extension_app_item.h
@@ -47,8 +47,7 @@ class ExtensionAppItem : public ChromeAppListItem,
// Reload the title and icon from the underlying extension.
void Reload();
- // Updates the app item's icon, if necessary adding an overlay and/or making
- // it gray.
+ // Updates the app item's icon, if necessary making it gray.
void UpdateIcon();
// Update page and app launcher ordinals to put the app in between |prev| and
@@ -75,9 +74,6 @@ class ExtensionAppItem : public ChromeAppListItem,
// Private equivalent to Activate(), without refocus for already-running apps.
void Launch(int event_flags);
- // Whether or not the app item needs an overlay.
- bool NeedsOverlay() const;
-
// Overridden from extensions::IconImage::Observer:
void OnExtensionIconImageChanged(extensions::IconImage* image) override;
void OnExtensionIconImageDestroyed(extensions::IconImage* image) override;
@@ -89,8 +85,6 @@ class ExtensionAppItem : public ChromeAppListItem,
// Overridden from AppListItem:
void Activate(int event_flags) override;
ui::MenuModel* GetContextMenuModel() override;
- // Updates the icon if the overlay needs to be added/removed.
- void OnExtensionPreferenceChanged() override;
const char* GetItemType() const override;
// Overridden from app_list::AppContextMenuDelegate:
@@ -110,9 +104,6 @@ class ExtensionAppItem : public ChromeAppListItem,
// Whether or not this app is a platform app.
bool is_platform_app_;
- // Whether this app item has an overlay.
- bool has_overlay_;
-
DISALLOW_COPY_AND_ASSIGN(ExtensionAppItem);
};
« no previous file with comments | « chrome/browser/ui/app_list/app_list_view_delegate.cc ('k') | chrome/browser/ui/app_list/extension_app_item.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698