Update SkSpecialImage API

src/core/SkSpecialImage.cpp

 /*
  * Copyright 2016 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file
  */
 
 #include "SkCanvas.h"
 #include "SkSpecialImage.h"
 #include "SkSpecialSurface.h"
 
 ///////////////////////////////////////////////////////////////////////////////
 class SkSpecialImage_Base : public SkSpecialImage {
 public:
-    SkSpecialImage_Base(const SkIRect& subset) : INHERITED(subset) { }
+    SkSpecialImage_Base(SkImageFilter::Proxy* proxy, const SkIRect& subset, uint32_t uniqueID)
+        : INHERITED(proxy, subset, uniqueID) {
+    }
     virtual ~SkSpecialImage_Base() { }
 
     virtual void onDraw(SkCanvas*, SkScalar x, SkScalar y, const SkPaint*) const = 0;
 
     virtual bool onPeekPixels(SkPixmap*) const { return false; }
 
     virtual GrTexture* onPeekTexture() const { return nullptr; }
 
+    // Delete this entry point ASAP

[Stephen White, 2016/02/17 21:44:02]

Log a bug, and reference it here? (see below)

[robertphillips, 2016/02/18 13:18:39]

Done.

+    virtual bool getBitmap(SkBitmap* result) const = 0;
+
     virtual SkSpecialSurface* onNewSurface(const SkImageInfo& info) const { return nullptr; }
 
 private:
     typedef SkSpecialImage INHERITED;
 };
 
 ///////////////////////////////////////////////////////////////////////////////
 static inline const SkSpecialImage_Base* as_IB(const SkSpecialImage* image) {
     return static_cast<const SkSpecialImage_Base*>(image);
 }
 
 void SkSpecialImage::draw(SkCanvas* canvas, SkScalar x, SkScalar y, const SkPaint* paint) const {
     return as_IB(this)->onDraw(canvas, x, y, paint);
 }
 
 bool SkSpecialImage::peekPixels(SkPixmap* pixmap) const {
     return as_IB(this)->onPeekPixels(pixmap);
 }
 
 GrTexture* SkSpecialImage::peekTexture() const {
     return as_IB(this)->onPeekTexture();
 }
 
 SkSpecialSurface* SkSpecialImage::newSurface(const SkImageInfo& info) const {
     return as_IB(this)->onNewSurface(info);
 }
 
+#if SK_SUPPORT_GPU
+#include "SkGr.h"
+#include "SkGrPixelRef.h"
+#endif
+
+SkSpecialImage* SkSpecialImage::internal_fromBM(SkImageFilter::Proxy* proxy,
+                                                const SkBitmap& src) {
+    // Need to test offset case!

[Stephen White, 2016/02/17 21:44:02]

You mean an offset in the pixelref? Log a bug maybe?

[robertphillips, 2016/02/18 13:18:39]

Right - Done.

+    if (src.getTexture()) {
+        return SkSpecialImage::NewFromGpu(proxy,
+                                          src.bounds(),
+                                          src.getGenerationID(),
+                                          src.getTexture());
+    }
+
+    return SkSpecialImage::NewFromRaster(proxy, src.bounds(), src);
+}
+
+bool SkSpecialImage::internal_getBM(SkBitmap* result) {
+    const SkSpecialImage_Base* ib = as_IB(this);
+
+    // TODO: need to test offset case!
+    return ib->getBitmap(result);
+}
+
+#include "SkBitmapDevice.h"
+#if SK_SUPPORT_GPU
+#include "SkGpuDevice.h"

[Stephen White, 2016/02/17 21:44:02]

Is this still needed? If not, remove it. If so, could it be in the existing #if
SK_SUPPORT_GPU clause below?

[robertphillips, 2016/02/18 13:18:39]

Done.

+#endif
+
+SkImageFilter::Proxy* SkSpecialImage::internal_getProxy() {
+    SkASSERT(fProxy);
+    return fProxy;
+}
+
 ///////////////////////////////////////////////////////////////////////////////
 #include "SkImage.h"
 #if SK_SUPPORT_GPU
-#include "SkGr.h"
 #include "SkGrPriv.h"
 #endif
 
 class SkSpecialImage_Image : public SkSpecialImage_Base {
 public:
-    SkSpecialImage_Image(const SkIRect& subset, const SkImage* image)
-        : INHERITED(subset)
+    SkSpecialImage_Image(SkImageFilter::Proxy* proxy, const SkIRect& subset, const SkImage* image)
+        : INHERITED(proxy, subset, image->uniqueID())
         , fImage(SkRef(image)) {
     }
 
     ~SkSpecialImage_Image() override { }
 
+    bool isOpaque() const override { return fImage->isOpaque(); }
+
+    size_t getSize() const override {
+        if (fImage->getTexture()) {
+            return fImage->getTexture()->gpuMemorySize();
+        } else {
+            SkImageInfo info;
+            size_t rowBytes;
+
+            if (fImage->peekPixels(&info, &rowBytes)) {
+                return info.height() * rowBytes;
+            }
+        }
+        return 0;
+    }
+
     void onDraw(SkCanvas* canvas, SkScalar x, SkScalar y, const SkPaint* paint) const override {
         SkRect dst = SkRect::MakeXYWH(x, y, this->subset().width(), this->subset().height());
 
         canvas->drawImageRect(fImage, this->subset(),
                               dst, paint, SkCanvas::kStrict_SrcRectConstraint);
     }
 
     bool onPeekPixels(SkPixmap* pixmap) const override {
         return fImage->peekPixels(pixmap);
     }
 
     GrTexture* onPeekTexture() const override { return fImage->getTexture(); }
 
+    bool getBitmap(SkBitmap* result) const override {
+        return false;
+    }
+
     SkSpecialSurface* onNewSurface(const SkImageInfo& info) const override {
 #if SK_SUPPORT_GPU
         GrTexture* texture = fImage->getTexture();
         if (texture) {
             GrSurfaceDesc desc = GrImageInfoToSurfaceDesc(info);
             desc.fFlags = kRenderTarget_GrSurfaceFlag;
 
-            return SkSpecialSurface::NewRenderTarget(texture->getContext(), desc);
+            return SkSpecialSurface::NewRenderTarget(this->proxy(), texture->getContext(), desc);
         }
 #endif
-        return SkSpecialSurface::NewRaster(info, nullptr);
+        return SkSpecialSurface::NewRaster(this->proxy(), info, nullptr);
     }
 
 private:
     SkAutoTUnref<const SkImage> fImage;
 
     typedef SkSpecialImage_Base INHERITED;
 };
 
 #ifdef SK_DEBUG
 static bool rect_fits(const SkIRect& rect, int width, int height) {
     return rect.fLeft >= 0 && rect.fLeft < width && rect.fLeft < rect.fRight &&
            rect.fRight >= 0 && rect.fRight <= width &&
            rect.fTop >= 0 && rect.fTop < height && rect.fTop < rect.fBottom &&
            rect.fBottom >= 0 && rect.fBottom <= height;
 }
 #endif
 
 SkSpecialImage* SkSpecialImage::NewFromImage(const SkIRect& subset, const SkImage* image) {
     SkASSERT(rect_fits(subset, image->width(), image->height()));
-    return new SkSpecialImage_Image(subset, image);
+    return new SkSpecialImage_Image(nullptr, subset, image);
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 #include "SkBitmap.h"
 #include "SkImageInfo.h"
 #include "SkPixelRef.h"
 
 class SkSpecialImage_Raster : public SkSpecialImage_Base {
 public:
-    SkSpecialImage_Raster(const SkIRect& subset, const SkBitmap& bm)
-        : INHERITED(subset)
+    SkSpecialImage_Raster(SkImageFilter::Proxy* proxy, const SkIRect& subset, const SkBitmap& bm)
+        : INHERITED(proxy, subset, bm.getGenerationID())
         , fBitmap(bm) {
         if (bm.pixelRef()->isPreLocked()) {
             // we only preemptively lock if there is no chance of triggering something expensive
             // like a lazy decode or imagegenerator. PreLocked means it is flat pixels already.
             fBitmap.lockPixels();
         }
     }
 
     ~SkSpecialImage_Raster() override { }
 
+    bool isOpaque() const override { return fBitmap.isOpaque(); }
+
+    size_t getSize() const override { return fBitmap.getSize(); }
+
     void onDraw(SkCanvas* canvas, SkScalar x, SkScalar y, const SkPaint* paint) const override {
         SkRect dst = SkRect::MakeXYWH(x, y,
                                       this->subset().width(), this->subset().height());
 
         canvas->drawBitmapRect(fBitmap, this->subset(),
                                dst, paint, SkCanvas::kStrict_SrcRectConstraint);
     }
 
     bool onPeekPixels(SkPixmap* pixmap) const override {
         const SkImageInfo info = fBitmap.info();
         if ((kUnknown_SkColorType == info.colorType()) || !fBitmap.getPixels()) {
             return false;
         }
         const void* pixels = fBitmap.getPixels();
         if (pixels) {
             if (pixmap) {
                 pixmap->reset(info, pixels, fBitmap.rowBytes());
             }
             return true;
         }
         return false;
     }
 
+    bool getBitmap(SkBitmap* result) const override {
+        *result = fBitmap;
+        return true;
+    }
+
     SkSpecialSurface* onNewSurface(const SkImageInfo& info) const override {
-        return SkSpecialSurface::NewRaster(info, nullptr);
+        return SkSpecialSurface::NewRaster(this->proxy(), info, nullptr);
     }
 
 private:
     SkBitmap fBitmap;
 
     typedef SkSpecialImage_Base INHERITED;
 };
 
-SkSpecialImage* SkSpecialImage::NewFromRaster(const SkIRect& subset, const SkBitmap& bm) {
+SkSpecialImage* SkSpecialImage::NewFromRaster(SkImageFilter::Proxy* proxy,
+                                              const SkIRect& subset,
+                                              const SkBitmap& bm) {
     SkASSERT(nullptr == bm.getTexture());
     SkASSERT(rect_fits(subset, bm.width(), bm.height()));
-    return new SkSpecialImage_Raster(subset, bm);
+    return new SkSpecialImage_Raster(proxy, subset, bm);
 }
 
 #if SK_SUPPORT_GPU
 ///////////////////////////////////////////////////////////////////////////////
 #include "GrTexture.h"
 
 class SkSpecialImage_Gpu : public SkSpecialImage_Base {
-public:
-    SkSpecialImage_Gpu(const SkIRect& subset, GrTexture* tex)
-        : INHERITED(subset)
-        , fTexture(SkRef(tex)) {
+public:                                       
+    SkSpecialImage_Gpu(SkImageFilter::Proxy* proxy, const SkIRect& subset,
+                       uint32_t uniqueID, GrTexture* tex, SkAlphaType at)
+        : INHERITED(proxy, subset, uniqueID)
+        , fTexture(SkRef(tex))
+        , fAlphaType(at) {
     }
 
     ~SkSpecialImage_Gpu() override { }
 
+    bool isOpaque() const override {
+        return GrPixelConfigIsOpaque(fTexture->config()) || fAlphaType == kOpaque_SkAlphaType;
+    }
+
+    size_t getSize() const override { return fTexture->gpuMemorySize(); }
+
     void onDraw(SkCanvas* canvas, SkScalar x, SkScalar y, const SkPaint* paint) const override {
         SkRect dst = SkRect::MakeXYWH(x, y,
                                       this->subset().width(), this->subset().height());
 
         SkBitmap bm;
 
-        static const bool kUnknownOpacity = false;
         GrWrapTextureInBitmap(fTexture,
-                              fTexture->width(), fTexture->height(), kUnknownOpacity, &bm);
+                              fTexture->width(), fTexture->height(), this->isOpaque(), &bm);
 
         canvas->drawBitmapRect(bm, this->subset(),
                                dst, paint, SkCanvas::kStrict_SrcRectConstraint);
     }
 
     GrTexture* onPeekTexture() const override { return fTexture; }
 
+    bool getBitmap(SkBitmap* result) const override {
+        const SkImageInfo info = GrMakeInfoFromTexture(fTexture, 
+                                                       this->width(), this->height(),
+                                                       this->isOpaque());
+        if (!result->setInfo(info)) {
+            return false;
+        }
+
+        result->setPixelRef(new SkGrPixelRef(info, fTexture))->unref();
+        return true;
+    }
+
     SkSpecialSurface* onNewSurface(const SkImageInfo& info) const override {
         GrSurfaceDesc desc = GrImageInfoToSurfaceDesc(info);
         desc.fFlags = kRenderTarget_GrSurfaceFlag;
 
-        return SkSpecialSurface::NewRenderTarget(fTexture->getContext(), desc);
+        return SkSpecialSurface::NewRenderTarget(this->proxy(), fTexture->getContext(), desc);
     }
 
 private:
     SkAutoTUnref<GrTexture> fTexture;
+    const SkAlphaType       fAlphaType;
 
     typedef SkSpecialImage_Base INHERITED;
 };
 
-SkSpecialImage* SkSpecialImage::NewFromGpu(const SkIRect& subset, GrTexture* tex) {
+SkSpecialImage* SkSpecialImage::NewFromGpu(SkImageFilter::Proxy* proxy,
+                                           const SkIRect& subset, 
+                                           uint32_t uniqueID,
+                                           GrTexture* tex,
+                                           SkAlphaType at) {
     SkASSERT(rect_fits(subset, tex->width(), tex->height()));
-    return new SkSpecialImage_Gpu(subset, tex);
+    return new SkSpecialImage_Gpu(proxy, subset, uniqueID, tex, at);
 }
 
 #else
 
-SkSpecialImage* SkSpecialImage::NewFromGpu(const SkIRect& subset, GrTexture* tex) {
+SkSpecialImage* SkSpecialImage::NewFromGpu(SkImageFilter::Proxy* proxy,
+                                           const SkIRect& subset,
+                                           uint32_t uniqueID,
+                                           GrTexture* tex,
+                                           SkAlphaType at) {
     return nullptr;
 }
 
 #endif