add readPixels to SkImage

src/image/SkImage.cpp

 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkBitmap.h"
 #include "SkCanvas.h"
 #include "SkImagePriv.h"
 #include "SkImage_Base.h"
+#include "SkReadPixelsRec.h"
 #include "SkSurface.h"
 
 uint32_t SkImage::NextUniqueID() {
     static int32_t gUniqueID;
 
     // never return 0;
     uint32_t id;
     do {
         id = sk_atomic_inc(&gUniqueID) + 1;
     } while (0 == id);
@@ skipping 14 matching lines @@
     size_t rowBytesStorage;
     if (NULL == info) {
         info = &infoStorage;
     }
     if (NULL == rowBytes) {
         rowBytes = &rowBytesStorage;
     }
     return as_IB(this)->onPeekPixels(info, rowBytes);
 }
 
-bool SkImage::readPixels(SkBitmap* bitmap, const SkIRect* subset) const {
-    if (NULL == bitmap) {
+bool SkImage::readPixels(const SkImageInfo& dstInfo, void* dstPixels, size_t dstRowBytes,
+                           int srcX, int srcY) const {
+    SkReadPixelsRec rec(dstInfo, dstPixels, dstRowBytes, srcX, srcY);
+    if (!rec.trim(this->width(), this->height())) {
         return false;
     }
-
-    SkIRect bounds = SkIRect::MakeWH(this->width(), this->height());
-
-    // trim against the bitmap, if its already been allocated
-    if (bitmap->pixelRef()) {
-        bounds.fRight = SkMin32(bounds.fRight, bitmap->width());
-        bounds.fBottom = SkMin32(bounds.fBottom, bitmap->height());
-        if (bounds.isEmpty()) {
-            return false;
-        }
-    }
-
-    if (subset && !bounds.intersect(*subset)) {
-        // perhaps we could return true + empty-bitmap?
-        return false;
-    }
-    return as_IB(this)->onReadPixels(bitmap, bounds);
+    return as_IB(this)->onReadPixels(rec.fInfo, rec.fPixels, rec.fRowBytes, rec.fX, rec.fY);
 }
 
 GrTexture* SkImage::getTexture() {
     return as_IB(this)->onGetTexture();
 }
 
 SkShader* SkImage::newShader(SkShader::TileMode tileX,
                              SkShader::TileMode tileY,
                              const SkMatrix* localMatrix) const {
     return as_IB(this)->onNewShader(tileX, tileY, localMatrix);
@@ skipping 23 matching lines @@
         case kRGB_565_SkColorType:
             return true;
         case kAlpha_8_SkColorType:
             return true;
         default:
             break;
     }
     return false;
 }
 
-bool SkImage_Base::onReadPixels(SkBitmap* bitmap, const SkIRect& subset) const {
-    if (bitmap->pixelRef()) {
-        const SkImageInfo info = bitmap->info();
-        if (kUnknown_SkColorType == info.colorType()) {
-            return false;
-        }
-        if (!raster_canvas_supports(info)) {
-            return false;
-        }
-    } else {
-        SkBitmap tmp;
-        if (!tmp.tryAllocN32Pixels(subset.width(), subset.height())) {
-            return false;
-        }
-        *bitmap = tmp;
+bool SkImage_Base::onReadPixels(const SkImageInfo& dstInfo, void* dstPixels, size_t dstRowBytes,
+                                int srcX, int srcY) const {
+    if (!raster_canvas_supports(dstInfo)) {
+        return false;
     }
 
-    SkRect srcR, dstR;
-    srcR.set(subset);
-    dstR = srcR;
-    dstR.offset(-dstR.left(), -dstR.top());
-
-    SkCanvas canvas(*bitmap);
+    SkBitmap bm;
+    bm.installPixels(dstInfo, dstPixels, dstRowBytes);
+    SkCanvas canvas(bm);
 
     SkPaint paint;
-    paint.setXfermodeMode(SkXfermode::kClear_Mode);
-    canvas.drawRect(dstR, paint);
+    paint.setXfermodeMode(SkXfermode::kSrc_Mode);
+    canvas.drawImage(this, -SkIntToScalar(srcX), -SkIntToScalar(srcY), &paint);
 
-    canvas.drawImageRect(this, &srcR, dstR);
     return true;
 }