Give Chrome Web Store app an icon in its manifest file.

chrome/browser/extensions/image_loading_tracker.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/image_loading_tracker.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/threading/sequenced_worker_pool.h"
 #include "chrome/browser/ui/webui/extensions/extension_icon_source.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_constants.h"
+#include "chrome/common/extensions/extension_file_util.h"
 #include "chrome/common/extensions/extension_resource.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_service.h"
 #include "grit/component_extension_resources_map.h"
 #include "grit/theme_resources.h"
 #include "skia/ext/image_operations.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/image/image_skia_rep.h"
 #include "webkit/glue/image_decoder.h"
 
 using content::BrowserThread;
 using extensions::Extension;
 
 namespace {
 
-struct ComponentExtensionResource {
-  const char* extension_id;
-  const int resource_id;
-};
-
-const ComponentExtensionResource kSpecialComponentExtensionResources[] = {
-  { extension_misc::kWebStoreAppId, IDR_WEBSTORE_ICON },
-  { extension_misc::kChromeAppId, IDR_PRODUCT_LOGO_128 },
-};
-
-// Finds special component extension resource id for given extension id.
-bool FindSpecialExtensionResourceId(const std::string& extension_id,
-                                    int* out_resource_id) {
-  for (size_t i = 0; i < arraysize(kSpecialComponentExtensionResources); ++i) {
-    if (extension_id == kSpecialComponentExtensionResources[i].extension_id) {
-      if (out_resource_id)
-        *out_resource_id = kSpecialComponentExtensionResources[i].resource_id;
-      return true;
-    }
-  }
-
-  return false;
-}
-
 bool ShouldResizeImageRepresentation(
     ImageLoadingTracker::ImageRepresentation::ResizeCondition resize_method,
     const gfx::Size& decoded_size,
     const gfx::Size& desired_size) {
   switch (resize_method) {
     case ImageLoadingTracker::ImageRepresentation::ALWAYS_RESIZE:
       return decoded_size != desired_size;
     case ImageLoadingTracker::ImageRepresentation::RESIZE_WHEN_LARGER:
       return decoded_size.width() > desired_size.width() ||
              decoded_size.height() > desired_size.height();
@@ skipping 172 matching lines @@
 
   // The thread that we need to call back on to report that we are done.
   BrowserThread::ID callback_thread_id_;
 
   DISALLOW_COPY_AND_ASSIGN(ImageLoader);
 };
 
 ////////////////////////////////////////////////////////////////////////////////
 // ImageLoadingTracker
 
-// static
-bool ImageLoadingTracker::IsSpecialBundledExtensionId(
-    const std::string& extension_id) {
-  int resource_id = -1;
-  return FindSpecialExtensionResourceId(extension_id, &resource_id);
-}
-
 ImageLoadingTracker::ImageLoadingTracker(Observer* observer)
     : observer_(observer),
       next_id_(0) {
   registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_UNLOADED,
                  content::NotificationService::AllSources());
 }
 
 ImageLoadingTracker::~ImageLoadingTracker() {
   // The loader is created lazily and is NULL if the tracker is destroyed before
   // any valid image load tasks have been posted.
@@ skipping 21 matching lines @@
   PendingLoadInfo load_info;
   load_info.extension = extension;
   load_info.cache = cache;
   load_info.extension_id = extension->id();
   load_info.pending_count = info_list.size();
   int id = next_id_++;
   load_map_[id] = load_info;
 
   for (std::vector<ImageRepresentation>::const_iterator it = info_list.begin();
        it != info_list.end(); ++it) {
-    int resource_id = -1;
-
-    // Load resources for special component extensions.
-    if (FindSpecialExtensionResourceId(load_info.extension_id, &resource_id)) {
-      if (!loader_)
-        loader_ = new ImageLoader(this);
-      loader_->LoadResource(*it, id, resource_id);
-      continue;
-    }
-
     // If we don't have a path we don't need to do any further work, just
     // respond back.
     if (it->resource.relative_path().empty()) {
       OnBitmapLoaded(NULL, *it, it->desired_size, id, false);
       continue;
     }
 
     DCHECK(extension->path() == it->resource.extension_root());
 
     // See if the extension has the bitmap already.
     if (extension->HasCachedImage(it->resource, it->desired_size)) {
       SkBitmap bitmap = extension->GetCachedImage(it->resource,
                                                   it->desired_size);
       OnBitmapLoaded(&bitmap, *it, it->desired_size, id, false);
       continue;
     }
 
     // Instruct the ImageLoader to load this on the File thread. LoadImage and
     // LoadResource do not block.
     if (!loader_)
       loader_ = new ImageLoader(this);
 
-    if (IsComponentExtensionResource(extension, it->resource, resource_id))
+    int resource_id = -1;
+    if (IsComponentExtensionResource(extension, it->resource.relative_path(),
+                                     &resource_id))
       loader_->LoadResource(*it, id, resource_id);
     else
       loader_->LoadImage(*it, id);
   }
 }
 
 bool ImageLoadingTracker::IsComponentExtensionResource(
     const Extension* extension,
-    const ExtensionResource& resource,
-    int& resource_id) const {
+    const FilePath& resource_path,
+    int* resource_id) {
+  static const GritResourceMap kExtraComponentExtensionResources[] = {
+    {"web_store/webstore_icon_128.png", IDR_WEBSTORE_ICON},
+    {"web_store/webstore_icon_16.png", IDR_WEBSTORE_ICON_16},
+    {"chrome_app/product_logo_128.png", IDR_PRODUCT_LOGO_128},
+    {"chrome_app/product_logo_16.png", IDR_PRODUCT_LOGO_16},
+  };
+  static const size_t kExtraComponentExtensionResourcesSize =
+      arraysize(kExtraComponentExtensionResources);
+
   if (extension->location() != Extension::COMPONENT)
     return false;
 
   FilePath directory_path = extension->path();
-  FilePath relative_path = directory_path.BaseName().Append(
-      resource.relative_path());
+  FilePath relative_path = directory_path.BaseName().Append(resource_path);
+  relative_path = relative_path.NormalizePathSeparators();
 
+  // TODO(tc): Make a map of FilePath -> resource ids so we don't have to
+  // covert to FilePaths all the time.  This will be more useful as we add
+  // more resources.
   for (size_t i = 0; i < kComponentExtensionResourcesSize; ++i) {
     FilePath resource_path =
         FilePath().AppendASCII(kComponentExtensionResources[i].name);
     resource_path = resource_path.NormalizePathSeparators();
 
     if (relative_path == resource_path) {
-      resource_id = kComponentExtensionResources[i].value;
+      *resource_id = kComponentExtensionResources[i].value;
       return true;
     }
   }
+  for (size_t i = 0; i < kExtraComponentExtensionResourcesSize; ++i) {
+    FilePath resource_path =
+        FilePath().AppendASCII(kExtraComponentExtensionResources[i].name);
+    resource_path = resource_path.NormalizePathSeparators();
+
+    if (relative_path == resource_path) {
+      *resource_id = kExtraComponentExtensionResources[i].value;
+      return true;
+    }
+  }
   return false;
 }
 
 void ImageLoadingTracker::OnBitmapLoaded(
     const SkBitmap* bitmap,
     const ImageRepresentation& image_info,
     const gfx::Size& original_size,
     int id,
     bool should_cache) {
   LoadMap::iterator load_map_it = load_map_.find(id);
@@ skipping 46 matching lines @@
   // Remove reference to this extension from all pending load entries. This
   // ensures we don't attempt to cache the bitmap when the load completes.
   for (LoadMap::iterator i = load_map_.begin(); i != load_map_.end(); ++i) {
     PendingLoadInfo* info = &i->second;
     if (info->extension == extension) {
       info->extension = NULL;
       info->cache = DONT_CACHE;
     }
   }
 }