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

Unified Diff: chrome/common/extensions/extension.cc

Issue 10843014: Generalize ExtensionIconSet to store icon paths for custom size sets (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: .. Created 8 years, 4 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/common/extensions/extension.cc
diff --git a/chrome/common/extensions/extension.cc b/chrome/common/extensions/extension.cc
index 4f3aaf7c94cda355d97901b7c8153717b59e785c..eafbc864460eac0dcf5b8603d9a419a9d1f307fd 100644
--- a/chrome/common/extensions/extension.cc
+++ b/chrome/common/extensions/extension.cc
@@ -1421,8 +1421,9 @@ bool Extension::LoadIcons(string16* error) {
return false;
}
- for (size_t i = 0; i < ExtensionIconSet::kNumIconSizes; ++i) {
- std::string key = base::IntToString(ExtensionIconSet::kIconSizes[i]);
+ for (size_t i = 0; i < extension_misc::kNumExtensionIconSizes; ++i) {
+ std::string key =
+ base::IntToString(extension_misc::kExtensionIconSizes[i]);
if (icons_value->HasKey(key)) {
std::string icon_path;
if (!icons_value->GetString(key, &icon_path)) {
@@ -1439,7 +1440,7 @@ bool Extension::LoadIcons(string16* error) {
errors::kInvalidIconPath, key);
return false;
}
- icons_.Add(ExtensionIconSet::kIconSizes[i], icon_path);
+ icons_.Add(extension_misc::kExtensionIconSizes[i], icon_path);
}
}
return true;
@@ -3014,25 +3015,25 @@ bool Extension::FormatPEMForFileOutput(const std::string& input,
// static
void Extension::DecodeIcon(const Extension* extension,
- ExtensionIconSet::Icons preferred_icon_size,
+ int preferred_icon_size,
ExtensionIconSet::MatchType match_type,
scoped_ptr<SkBitmap>* result) {
std::string path = extension->icons().Get(preferred_icon_size, match_type);
- ExtensionIconSet::Icons size = extension->icons().GetIconSizeFromPath(path);
+ int size = extension->icons().GetIconSizeFromPath(path);
ExtensionResource icon_resource = extension->GetResource(path);
DecodeIconFromPath(icon_resource.GetFilePath(), size, result);
}
// static
void Extension::DecodeIcon(const Extension* extension,
- ExtensionIconSet::Icons icon_size,
+ int icon_size,
scoped_ptr<SkBitmap>* result) {
DecodeIcon(extension, icon_size, ExtensionIconSet::MATCH_EXACTLY, result);
}
// static
void Extension::DecodeIconFromPath(const FilePath& icon_path,
- ExtensionIconSet::Icons icon_size,
+ int icon_size,
scoped_ptr<SkBitmap>* result) {
if (icon_path.empty())
return;

Powered by Google App Engine
This is Rietveld 408576698