chromeos: Fix pixelated icons in app list and launcher (part 2) (by xiyuan)

chrome/browser/extensions/image_loading_tracker.h

Index: chrome/browser/extensions/image_loading_tracker.h
diff --git a/chrome/browser/extensions/image_loading_tracker.h b/chrome/browser/extensions/image_loading_tracker.h
index b163651e9a551d4ae3b53533f4596a181d350ea6..645d31541295aadd7184589b02a76c772d9acc5d 100644
--- a/chrome/browser/extensions/image_loading_tracker.h
+++ b/chrome/browser/extensions/image_loading_tracker.h
@@ -6,13 +6,18 @@
 #define CHROME_BROWSER_EXTENSIONS_IMAGE_LOADING_TRACKER_H_
 
 #include <map>
+#include <string>
+#include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/ref_counted.h"
+#include "chrome/common/extensions/extension_icon_set.h"
 #include "chrome/common/extensions/extension_resource.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
+#include "ui/base/layout.h"
+#include "ui/gfx/image/image_skia.h"
 #include "ui/gfx/size.h"
 
 class SkBitmap;
@@ -35,7 +40,7 @@ class Image;
 // Observer::OnImageLoaded and call:
 //   tracker_.LoadImage(extension, resource, max_size, false);
 // ... and wait for OnImageLoaded to be called back on you with a pointer to the
-// SkBitmap loaded.
+// ImageSkia loaded.
 // NOTE: if the image is available already (or the resource is not valid), the
 // Observer is notified immediately from the call to LoadImage. In other words,
 // by the time LoadImage returns the observer has been notified.
@@ -62,23 +67,49 @@ class ImageLoadingTracker : public content::NotificationObserver {
     virtual ~Observer();
   };
 
-  // Information about a single image to load from a extension resource.
-  struct ImageInfo {
-    ImageInfo(const ExtensionResource& resource, gfx::Size max_size);
-    ~ImageInfo();
+  // Information about a single bitmap/image rep to load from an extension

[Aaron Boodman, 2012/08/02 17:50:56]

When you're dealing with ImageRepInfo (or ImageRepresentation as I suggest
below), just say 'image representation' in the comments. The fact that we are
translating bitmaps into ImageRepresentations is an implementation detail
elsewhere that we don't need to spread around.

[tbarzic, 2012/08/02 20:12:52]

Done.

+  // resource.
+  struct ImageRepInfo {

[Aaron Boodman, 2012/08/02 17:50:56]

per styleguide [1]
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Genera...,
we are supposed to eschew abbreviation. How about just ImageRepresentation?

[tbarzic, 2012/08/02 20:12:52]

Done.

+    // Enum values to indicate whether to resize loaded bitmap when it is larger
+    // than |desired_size| or always resize it.
+    enum ResizeMethod {
+      RESIZE_WHEN_LARGER,
+      ALWAYS_RESIZE,
+    };
+
+    ImageRepInfo(const ExtensionResource& resource,
+                 ResizeMethod resize_method,
+                 const gfx::Size& desired_size,
+                 ui::ScaleFactor scale_factor);
+    ~ImageRepInfo();
+
+    // Extension resource to load.
     ExtensionResource resource;
-    // If the loaded image is larger than |max_size| it will be resized to those
-    // dimensions.
-    gfx::Size max_size;
+
+    ResizeMethod resize_method;
+
+    // When |resize_method| is ALWAYS_RESIZE or when the loaded image is larger
+    // than |desired_size| it will be resized to these dimensions.
+    gfx::Size desired_size;
+
+    // |scale_factor| is used to construct the loaded gfx::ImageSkia.
+    ui::ScaleFactor scale_factor;
   };
 
   explicit ImageLoadingTracker(Observer* observer);
   virtual ~ImageLoadingTracker();
 
+  // Checks whether given extension resource could be loaded.
+  static bool CanLoadImage(const extensions::Extension* extension,
+                           const ExtensionResource& resource);
+
   // Specify image resource to load. If the loaded image is larger than
   // |max_size| it will be resized to those dimensions. IMPORTANT NOTE: this
   // function may call back your observer synchronously (ie before it returns)
   // if the image was found in the cache.
+  // Note this method loads a raw bitmap from the resource. All sizes given are
+  // assumed to be in pixels. If you want to load a DIP-aware image, use
+  // |LoadImageSkia| instead.
   void LoadImage(const extensions::Extension* extension,
                  const ExtensionResource& resource,
                  const gfx::Size& max_size,
@@ -88,7 +119,7 @@ class ImageLoadingTracker : public content::NotificationObserver {
   // extension. This is used to load multiple resolutions of the same image
   // type.
   void LoadImages(const extensions::Extension* extension,
-                  const std::vector<ImageInfo>& info_list,
+                  const std::vector<ImageRepInfo>& info_list,
                   CacheParam cache);
 
   // Returns the ID used for the next image that is loaded. That is, the return
@@ -97,18 +128,19 @@ class ImageLoadingTracker : public content::NotificationObserver {
   int next_id() const { return next_id_; }
 
  private:
-  // Information for pending image load operation for one or more images.
+  // Information for pending resource load operation for one or more
+  // bitmaps/image reps.

[Aaron Boodman, 2012/08/02 17:50:56]

Same thing here with 'bitmap/', and throughout.

[tbarzic, 2012/08/02 20:12:52]

Done.

   struct PendingLoadInfo {
     PendingLoadInfo();
     ~PendingLoadInfo();
 
     const extensions::Extension* extension;
-    // This is cached separate from |extension| in case the extension in
+    // This is cached separate from |extension| in case the extension is
     // unloaded.
     std::string extension_id;
     CacheParam cache;
     size_t pending_count;
-    std::vector<SkBitmap> bitmaps;
+    gfx::ImageSkia image_skia;
   };
 
   // Maps an integer identifying a load request to a PendingLoadInfo.
@@ -116,14 +148,18 @@ class ImageLoadingTracker : public content::NotificationObserver {
 
   class ImageLoader;
 
-  // When an image has finished loaded and been resized on the file thread, it
+  // When a bitmap has finished loading and been resized on the file thread, it

[Aaron Boodman, 2012/08/02 17:50:56]

This comment contains way too much implementation detail. It can pretty much
just say 'Called on the UI thread when a bitmap has finished loading.'

[tbarzic, 2012/08/02 20:12:52]

agree

   // is posted back to this method on the original thread.  This method then
   // calls the observer's OnImageLoaded and deletes the ImageLoadingTracker if
-  // it was the last image in the list. The |original_size| should be the size
-  // of the image before any resizing was done.
-  // |image| may be null if the file failed to decode.
-  void OnImageLoaded(SkBitmap* image, const ExtensionResource& resource,
-                     const gfx::Size& original_size, int id, bool should_cache);
+  // the caller will not request image reps for aditional scale factors from the
+  // ImageLoadingTracker. The |original_size| should be the size of the bitmap
+  // before any resizing was done.
+  // |bitmap| may be null if the file failed to decode.
+  void OnBitmapLoaded(SkBitmap* bitmap,
+                      const ImageRepInfo& image_info,
+                      const gfx::Size& original_size,
+                      int id,
+                      bool should_cache);
 
   // Checks whether image is a component extension resource. Returns false
   // if a given |resource| does not have a corresponding image in bundled