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

Unified Diff: chrome/browser/extensions/extension_uninstall_dialog.cc

Issue 9428025: Add support for multiple icon sizes for Mac platform apps (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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_uninstall_dialog.cc
diff --git a/chrome/browser/extensions/extension_uninstall_dialog.cc b/chrome/browser/extensions/extension_uninstall_dialog.cc
index d5bed37619d2d7ddc38f2f8aa34cc49e10a44350..276e818fdddec97f20bb88e584df2977aab9f4ab 100644
--- a/chrome/browser/extensions/extension_uninstall_dialog.cc
+++ b/chrome/browser/extensions/extension_uninstall_dialog.cc
@@ -13,6 +13,7 @@
#include "grit/generated_resources.h"
#include "grit/theme_resources.h"
#include "ui/base/resource/resource_bundle.h"
+#include "ui/gfx/image/image.h"
// Size of extension icon in top left of dialog.
static const int kIconSize = 69;
@@ -41,9 +42,9 @@ void ExtensionUninstallDialog::ConfirmUninstall(const Extension* extension) {
ImageLoadingTracker::DONT_CACHE);
}
-void ExtensionUninstallDialog::SetIcon(SkBitmap* image) {
+void ExtensionUninstallDialog::SetIcon(const gfx::Image* image) {
if (image)
- icon_ = *image;
+ icon_ = *image->ToSkBitmap();
else
icon_ = SkBitmap();
if (icon_.empty()) {
@@ -57,8 +58,8 @@ void ExtensionUninstallDialog::SetIcon(SkBitmap* image) {
}
}
-void ExtensionUninstallDialog::OnImageLoaded(SkBitmap* image,
- const ExtensionResource& resource,
+void ExtensionUninstallDialog::OnImageLoaded(const gfx::Image* image,
+ const std::string& extension_id,
int index) {
SetIcon(image);

Powered by Google App Engine
This is Rietveld 408576698