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

Unified Diff: chrome/browser/extensions/extension_install_ui.h

Issue 9586018: Add support for multiple icon sizes for Mac platform apps (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix build Created 8 years, 10 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/extensions/extension_install_ui.h
diff --git a/chrome/browser/extensions/extension_install_ui.h b/chrome/browser/extensions/extension_install_ui.h
index bb3ad1d83c3ef092146ea2371674ff9736b1ace3..9a35da09e6e717f9e31feabd8b10542623aed284 100644
--- a/chrome/browser/extensions/extension_install_ui.h
+++ b/chrome/browser/extensions/extension_install_ui.h
@@ -174,8 +174,9 @@ class ExtensionInstallUI : public ImageLoadingTracker::Observer {
virtual void OnInstallFailure(const string16& error);
// ImageLoadingTracker::Observer:
- virtual void OnImageLoaded(
- SkBitmap* image, const ExtensionResource& resource, int index) OVERRIDE;
+ virtual void OnImageLoaded(const gfx::Image& image,
+ const std::string& extension_id,
+ int index) OVERRIDE;
// Opens a new tab page and animates the app icon for the app with id
// |app_id|.
@@ -201,7 +202,7 @@ class ExtensionInstallUI : public ImageLoadingTracker::Observer {
// Sets the icon that will be used in any UI. If |icon| is NULL, or contains
// an empty bitmap, then a default icon will be used instead.
- void SetIcon(SkBitmap* icon);
+ void SetIcon(const SkBitmap* icon);
// Starts the process of showing a confirmation UI, which is split into two.
// 1) Set off a 'load icon' task.
« no previous file with comments | « chrome/browser/extensions/extension_icon_manager_unittest.cc ('k') | chrome/browser/extensions/extension_install_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698