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(const SkIRect& subset, uint32_t uniqueID) : INHERITED(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
+    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(const SkBitmap& src) {
+    // Need to test offset case!
+    if (src.getTexture()) {
+        return SkSpecialImage::NewFromGpu(src.bounds(), src.getGenerationID(), src.getTexture());
+    }
+
+    return SkSpecialImage::NewFromRaster(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"
+#endif
+
+SkImageFilter::Proxy* SkSpecialImage::internal_getProxy() {
+    if (fProxy) {
+        SkASSERT(fDevice);
+        return fProxy;
+    }
+
+    const SkImageInfo info = SkImageInfo::MakeN32Premul(16, 16);
+    const SkSurfaceProps props(SkSurfaceProps::kLegacyFontHost_InitType);
+
+#if SK_SUPPORT_GPU
+    if (this->peekTexture()) {
+        GrContext* context = this->peekTexture()->getContext();
+
+        // Youch!
+        fDevice.reset(SkGpuDevice::Create(context, SkSurface::kYes_Budgeted, info, 0, &props,

[Stephen White, 2016/02/17 17:00:02]

This doesn't make sense to me. Why do we need to have a per-SkSpecialImage
proxy? It shouldn't need its own storage. Can't we just have a single proxy
that's copied down the chain, or something? I.e., the calling device creates
one, and sets in the primitive's SkSpecialImage, and then the base-class
onFilterImage() (the non-deprecated one) pulls it out and puts it in the surface
it creates? Something like that? So that it can be propagated to the created
SkSpecialImages?

Alternatively, we could make a new proxy that contains the SkSpecialImage, and
implements createDevice() by calling the image's surface creation functions. Not
sure if you can actually extract the device out of that, though.

[robertphillips, 2016/02/17 19:10:28]

Done.

+                                          SkGpuDevice::kUninit_InitContents));
+
+        fProxy = new SkImageFilter::DeviceProxy(fDevice);
+        return fProxy;
+    }
+#endif
+
+    fDevice.reset(SkBitmapDevice::Create(info, props));
+
+    fProxy = new SkImageFilter::DeviceProxy(fDevice);
+    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)
+        : INHERITED(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);
         }
 #endif
@@ skipping 21 matching lines @@
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 #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)
+        : INHERITED(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);
     }
 
 private:
     SkBitmap fBitmap;
 
     typedef SkSpecialImage_Base INHERITED;
 };
 
 SkSpecialImage* SkSpecialImage::NewFromRaster(const SkIRect& subset, const SkBitmap& bm) {
     SkASSERT(nullptr == bm.getTexture());
     SkASSERT(rect_fits(subset, bm.width(), bm.height()));
     return new SkSpecialImage_Raster(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(const SkIRect& subset, uint32_t uniqueID, GrTexture* tex, SkAlphaType at)
+        : INHERITED(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);
     }
 
 private:
     SkAutoTUnref<GrTexture> fTexture;
+    const SkAlphaType       fAlphaType;
 
     typedef SkSpecialImage_Base INHERITED;
 };
 
-SkSpecialImage* SkSpecialImage::NewFromGpu(const SkIRect& subset, GrTexture* tex) {
+SkSpecialImage* SkSpecialImage::NewFromGpu(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(subset, uniqueID, tex, at);
 }
 
 #else
 
-SkSpecialImage* SkSpecialImage::NewFromGpu(const SkIRect& subset, GrTexture* tex) {
+SkSpecialImage* SkSpecialImage::NewFromGpu(const SkIRect& subset,
+                                           uint32_t uniqueID,
+                                           GrTexture* tex,
+                                           SkAlphaType at) {
     return nullptr;
 }
 
 #endif