Makes ImageSkia more like SkBitmap

ui/gfx/image/image_skia.h

Index: ui/gfx/image/image_skia.h
diff --git a/ui/gfx/image/image_skia.h b/ui/gfx/image/image_skia.h
index bdfb53f8d89ae941d143d303b88c35a41af76551..21f72a81f14c924926aae487db0203bda9b946a5 100644
--- a/ui/gfx/image/image_skia.h
+++ b/ui/gfx/image/image_skia.h
@@ -9,80 +9,103 @@
 #include <vector>
 
 #include "base/basictypes.h"
+#include "base/memory/ref_counted.h"
+#include "ui/base/ui_export.h"
 #include "third_party/skia/include/core/SkBitmap.h"
-#include "ui/gfx/canvas.h"
-#include "ui/gfx/size.h"
 
 namespace gfx {
 
+namespace internal {
+class ImageSkiaStorage;
+}  // namespace internal
+
 // Container for the same image at different densities, similar to NSImage.
 // Smallest image is assumed to represent 1x density.
-
-// ImageSkia should be used for caching and drawing images of different
-// densities. It should not be used as an SkBitmap wrapper.
+//
+// ImageSkia should be used whenever possible instead of SkBitmap.
+// Functions which mutate the image should operate on the SkBitmap returned
+// from ImageSkia::GetBitmapForScale, not on ImageSkia.
+//
+// ImageSkia supports copy semantics as legacy from SkBitmap. Ideally, this
+// would not be necessary and all classes which add a ref to ImageSkia would
+// hold a ref_ptr.
 class UI_EXPORT ImageSkia {
  public:
+  // Creates null instance.
+  ImageSkia();
+
+  // Adds ref to passed in bitmap.
+  // DIP width and height are set based on primary monitor scale factor.
+  // TODO(pkotwicz): Remove this constructor.
+  ImageSkia(const SkBitmap& bitmap);
+
+  // Adds ref to passed in bitmap.
+  // DIP width and height are set based on |scale_factor|.

[oshima, 2012/04/30 21:10:08]

My understanding is that this creates ImageSkia with one bitmap that is desgined
for |scale_factor|. Am I wrong?

+  ImageSkia(const SkBitmap& bitmap, float scale_factor);
+
+  // Takes ownership of passed in bitmap.
+  // DIP width and height are set based on primary monitor scale factor.
   explicit ImageSkia(const SkBitmap* bitmap);
+
+  // Takes ownership of passed in bitmap.
+  // DIP width and height are set assuming smallest bitmap has scale factor of
+  // 1x.
   explicit ImageSkia(const std::vector<const SkBitmap*>& bitmaps);
-  ~ImageSkia();
 
-  // Build mipmap at time of next call to |DrawToCanvasInt|.
-  void BuildMipMap();
-
-  // Draws the image with the origin at the specified location. The upper left
-  // corner of the image is rendered at the specified location.
-  void DrawToCanvasInt(Canvas* canvas, int x, int y);
-
-  // Draws the image with the origin at the specified location, using the
-  // specified paint. The upper left corner of the image is rendered at the
-  // specified location.
-  void DrawToCanvasInt(Canvas* canvas,
-                       int x, int y,
-                       const SkPaint& paint);
-
-  // Draws a portion of the image in the specified location. The src parameters
-  // correspond to the region of the image to draw in the region defined
-  // by the dest coordinates.
-  //
-  // If the width or height of the source differs from that of the destination,
-  // the image will be scaled. When scaling down, it is highly recommended
-  // that you call BuildMipMap() on your image to ensure that it has
-  // a mipmap, which will result in much higher-quality output. Set |filter| to
-  // use filtering for bitmaps, otherwise the nearest-neighbor algorithm is used
-  // for resampling.
-  //
-  // An optional custom SkPaint can be provided.
-  void DrawToCanvasInt(Canvas* canvas,
-                       int src_x, int src_y, int src_w, int src_h,
-                       int dest_x, int dest_y, int dest_w, int dest_h,
-                       bool filter);
-  void DrawToCanvasInt(Canvas* canvas,
-                       int src_x, int src_y, int src_w, int src_h,
-                       int dest_x, int dest_y, int dest_w, int dest_h,
-                       bool filter,
-                       const SkPaint& paint);
-
-  // Returns true if |size_| is empty.
-  bool IsZeroSized() const { return size_.IsEmpty(); }
+  // Copies a reference to |other|'s storage.
+  // TODO(pkotwicz): Remove this method.
+  ImageSkia(const ImageSkia& other);
 
-  // Width and height of image in DIP coordinate system.
-  int width() const { return size_.width(); }
-  int height() const { return size_.height(); }
+  // Copies a reference to |other|'s storage.
+  ImageSkia& operator=(const ImageSkia& other);
 
-  // Returns a vector with the SkBitmaps contained in this object.
-  const std::vector<const SkBitmap*>& bitmaps() const { return bitmaps_; }
+  // Converts from SkBitmap.
+  // Adds ref to passed in bitmap.
+  // DIP width and height are set based on primary monitor scale factor.
+  ImageSkia& operator=(const SkBitmap& other);
+
+  // Converts to SkBitmap.
+  // TODO(pkotwicz): Remove this function.
+  operator SkBitmap() const;
+
+  ~ImageSkia();
 
- private:
   // Returns the bitmap whose density best matches |x_scale_factor| and
   // |y_scale_factor|.
   const SkBitmap* GetBitmapForScale(float x_scale_factor,
                                     float y_scale_factor) const;
 
-  std::vector<const SkBitmap*> bitmaps_;
-  gfx::Size size_;
-  bool mip_map_build_pending_;
+  // Returns true if object is null or |size_| is empty.
+  bool empty() const;
+
+  // Returns true if this is a null object.
+  // TODO(pkotwicz): Merge this function into empty().
+  bool isNull() const { return storage_ == NULL; }
+
+  // Width and height of image in DIP coordinate system.
+  int width() const;
+  int height() const;
+
+  // Wrapper function for SkBitmap extractBitmap.
+  // Operates on bitmap at index 0 if available.
+  // TODO(pkotwicz): Remove this function from gfx::ImageSkia.
+  bool extractSubset(ImageSkia* dst, SkIRect& subset) const;
+
+  // Returns a vector with the SkBitmaps contained in this object.
+  const std::vector<const SkBitmap*>& bitmaps() const;
+
+ private:
+  // Initialize ImageStorage with passed in parameters.
+  // If |bitmap.isNull()|, ImageStorage is set to NULL.
+  // Scale factor if is set based on primary monitor scale factor.
+  void Init(const SkBitmap* bitmap);
+
+  // Initialize ImageStorage with passed in parameters.
+  // If |bitmap.isNull()|, ImageStorage is set to NULL.
+  void Init(const SkBitmap* bitmap, float scale_factor);
 
-  DISALLOW_COPY_AND_ASSIGN(ImageSkia);
+  // A refptr so that ImageRepSkia can be copied cheaply.
+  scoped_refptr<internal::ImageSkiaStorage> storage_;
 };
 
 }  // namespace gfx