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

Unified Diff: chrome/browser/extensions/api/extension_action/extension_actions_api.cc

Issue 11017046: Use LANCZOS3 resize algorithm to generate missing image reps for extension action icons. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 2 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/api/extension_action/extension_actions_api.cc
diff --git a/chrome/browser/extensions/api/extension_action/extension_actions_api.cc b/chrome/browser/extensions/api/extension_action/extension_actions_api.cc
index 3b7a2b496747281dec1cf95d8f53e664872284a6..195a623a8443959e04bbe6efe7c743a4669de452 100644
--- a/chrome/browser/extensions/api/extension_action/extension_actions_api.cc
+++ b/chrome/browser/extensions/api/extension_action/extension_actions_api.cc
@@ -27,6 +27,7 @@
#include "chrome/common/render_messages.h"
#include "content/public/browser/navigation_entry.h"
#include "content/public/browser/notification_service.h"
+#include "ui/gfx/image/image_skia_operations.h"
namespace {
@@ -130,7 +131,11 @@ void SetDefaultsFromValue(const base::DictionaryValue* dict,
icon.AddRepresentation(gfx::ImageSkiaRep(bitmap, kIconSizes[i].scale));
}
}
- action->SetIcon(kTabId, gfx::Image(icon));
+
+ gfx::ImageSkia final_icon =
+ gfx::ImageSkiaOperations::CreateImageWithFallbackResizeMethod(
+ icon, skia::ImageOperations::RESIZE_LANCZOS3);
+ action->SetIcon(kTabId, gfx::Image(final_icon));
}
}
@@ -469,7 +474,10 @@ bool ExtensionActionSetIconFunction::RunExtensionAction() {
}
}
- extension_action_->SetIcon(tab_id_, gfx::Image(icon));
+ gfx::ImageSkia final_icon =
+ gfx::ImageSkiaOperations::CreateImageWithFallbackResizeMethod(
+ icon, skia::ImageOperations::RESIZE_LANCZOS3);
+ extension_action_->SetIcon(tab_id_, gfx::Image(final_icon));
} else if (details_->GetInteger("iconIndex", &icon_index)) {
// Obsolete argument: ignore it.
return true;

Powered by Google App Engine
This is Rietveld 408576698