Make filters use SkImage instead of SkBitmap

src/effects/SkXfermodeImageFilter.cpp

 /*
  * Copyright 2013 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkXfermodeImageFilter.h"
 #include "SkCanvas.h"
-#include "SkDevice.h"
+#include "SkSurface.h"
 #include "SkColorPriv.h"
 #include "SkReadBuffer.h"
 #include "SkWriteBuffer.h"
 #include "SkXfermode.h"
 #if SK_SUPPORT_GPU
 #include "GrContext.h"
 #include "effects/GrSimpleTextureEffect.h"
 #include "SkGr.h"
 #endif
 
@@ skipping 16 matching lines @@
     SkAutoTUnref<SkXfermode> mode(buffer.readXfermode());
     return Create(mode, common.getInput(0), common.getInput(1), &common.cropRect(), common.uniqueID());
 }
 
 void SkXfermodeImageFilter::flatten(SkWriteBuffer& buffer) const {
     this->INHERITED::flatten(buffer);
     buffer.writeFlattenable(fMode);
 }
 
 bool SkXfermodeImageFilter::onFilterImage(Proxy* proxy,
-                                            const SkBitmap& src,
-                                            const Context& ctx,
-                                            SkBitmap* dst,
-                                            SkIPoint* offset) const {
-    SkBitmap background = src, foreground = src;
+                                          SkImage& src,
+                                          const Context& ctx,
+                                          SkAutoTUnref<SkImage>& dst,
+                                          SkIPoint* offset) const {
+    SkAutoTUnref<SkImage> background(SkRef(&src));
+    SkAutoTUnref<SkImage> foreground(SkRef(&src));
     SkImageFilter* backgroundInput = getInput(0);
     SkImageFilter* foregroundInput = getInput(1);
     SkIPoint backgroundOffset = SkIPoint::Make(0, 0);
+    //TODO MAKE APPLY CROP RECT PTR TO HANDLE NULL.
     if (backgroundInput &&
-        !backgroundInput->filterImage(proxy, src, ctx, &background, &backgroundOffset)) {
-        background.reset();
+        !backgroundInput->filterImage(proxy, src, ctx, background, &backgroundOffset)) {
+        background.reset(NULL);
     }
     SkIPoint foregroundOffset = SkIPoint::Make(0, 0);
     if (foregroundInput &&
-        !foregroundInput->filterImage(proxy, src, ctx, &foreground, &foregroundOffset)) {
-        foreground.reset();
+        !foregroundInput->filterImage(proxy, src, ctx, foreground, &foregroundOffset)) {
+        foreground.reset(NULL);
     }
 
     SkIRect bounds, foregroundBounds;
-    if (!applyCropRect(ctx, foreground, foregroundOffset, &foregroundBounds)) {
+    if (!applyCropRect(ctx, *foreground, foregroundOffset, &foregroundBounds)) {
         foregroundBounds.setEmpty();
-        foreground.reset();
+        foreground.reset(NULL);
     }
-    if (!applyCropRect(ctx, background, backgroundOffset, &bounds)) {
+    if (!applyCropRect(ctx, *background, backgroundOffset, &bounds)) {
         bounds.setEmpty();
-        background.reset();
+        background.reset(NULL);
     }
     bounds.join(foregroundBounds);
     if (bounds.isEmpty()) {
         return false;
     }
 
-    SkAutoTUnref<SkBaseDevice> device(proxy->createDevice(bounds.width(), bounds.height()));
-    if (NULL == device.get()) {
+    SkAutoTUnref<SkSurface> surface(proxy->createSurface(bounds.width(), bounds.height()));
+    if (NULL == surface) {
         return false;
     }
-    SkCanvas canvas(device);
-    canvas.translate(SkIntToScalar(-bounds.left()), SkIntToScalar(-bounds.top()));
+    SkCanvas* canvas = surface->getCanvas();
+    canvas->translate(SkIntToScalar(-bounds.left()), SkIntToScalar(-bounds.top()));
     SkPaint paint;
     paint.setXfermodeMode(SkXfermode::kSrc_Mode);
-    canvas.drawBitmap(background, SkIntToScalar(backgroundOffset.fX),
-                      SkIntToScalar(backgroundOffset.fY), &paint);
+    canvas->drawImage(background, SkIntToScalar(backgroundOffset.fX),
+                     SkIntToScalar(backgroundOffset.fY), &paint);
     paint.setXfermode(fMode);
-    canvas.drawBitmap(foreground, SkIntToScalar(foregroundOffset.fX),
+    canvas->drawImage(foreground, SkIntToScalar(foregroundOffset.fX),
                       SkIntToScalar(foregroundOffset.fY), &paint);
-    canvas.clipRect(SkRect::Make(foregroundBounds), SkRegion::kDifference_Op);
+    canvas->clipRect(SkRect::Make(foregroundBounds), SkRegion::kDifference_Op);
     paint.setColor(SK_ColorTRANSPARENT);
-    canvas.drawPaint(paint);
-    *dst = device->accessBitmap(false);
+    canvas->drawPaint(paint);
+
+    foreground.reset(NULL);
+    background.reset(NULL);
+
+    SkImage* image = surface->newImageSnapshot(SkSurface::kYes_Budgeted);
+    if (NULL == image) {
+        return false;
+    }
+    dst.reset(image);
+
     offset->fX = bounds.left();
     offset->fY = bounds.top();
     return true;
 }
 
 #ifndef SK_IGNORE_TO_STRING
 void SkXfermodeImageFilter::toString(SkString* str) const {
     str->appendf("SkXfermodeImageFilter: (");
     str->appendf("xfermode: (");
     if (fMode) {
         fMode->toString(str);
     }
     str->append("))");
 }
 #endif
 
 #if SK_SUPPORT_GPU
 
 bool SkXfermodeImageFilter::canFilterImageGPU() const {
     return fMode && fMode->asFragmentProcessor(NULL, NULL) && !cropRectIsSet();
 }
 
 bool SkXfermodeImageFilter::filterImageGPU(Proxy* proxy,
-                                           const SkBitmap& src,
+                                           SkImage& src,
                                            const Context& ctx,
-                                           SkBitmap* result,
+                                           SkAutoTUnref<SkImage>& result,
                                            SkIPoint* offset) const {
-    SkBitmap background = src;
+    SkAutoTUnref<SkImage> background(SkRef(&src));
     SkIPoint backgroundOffset = SkIPoint::Make(0, 0);
-    if (getInput(0) && !getInput(0)->getInputResultGPU(proxy, src, ctx, &background,
+    if (getInput(0) && !getInput(0)->getInputResultGPU(proxy, src, ctx, background,
                                                        &backgroundOffset)) {
         return onFilterImage(proxy, src, ctx, result, offset);
     }
-    GrTexture* backgroundTex = background.getTexture();
+    GrTexture* backgroundTex = background->getTexture();
 
     if (NULL == backgroundTex) {
         SkASSERT(false);
         return false;
     }
 
-    SkBitmap foreground = src;
+    SkAutoTUnref<SkImage> foreground(SkRef(&src));
     SkIPoint foregroundOffset = SkIPoint::Make(0, 0);
-    if (getInput(1) && !getInput(1)->getInputResultGPU(proxy, src, ctx, &foreground,
+    if (getInput(1) && !getInput(1)->getInputResultGPU(proxy, src, ctx, foreground,
                                                        &foregroundOffset)) {
         return onFilterImage(proxy, src, ctx, result, offset);
     }
-    GrTexture* foregroundTex = foreground.getTexture();
+    GrTexture* foregroundTex = foreground->getTexture();
     GrContext* context = foregroundTex->getContext();
 
     GrFragmentProcessor* xferProcessor = NULL;
 
     GrSurfaceDesc desc;
     desc.fFlags = kRenderTarget_GrSurfaceFlag | kNoStencil_GrSurfaceFlag;
     desc.fWidth = src.width();
     desc.fHeight = src.height();
     desc.fConfig = kSkia8888_GrPixelConfig;
     SkAutoTUnref<GrTexture> dst(
         context->refScratchTexture(desc, GrContext::kApprox_ScratchTexMatch));
     if (!dst) {
         return false;
     }
     GrContext::AutoRenderTarget art(context, dst->asRenderTarget());
 
     if (!fMode || !fMode->asFragmentProcessor(&xferProcessor, backgroundTex)) {
         // canFilterImageGPU() should've taken care of this
         SkASSERT(false);
         return false;
     }
 
     SkMatrix foregroundMatrix = GrCoordTransform::MakeDivByTextureWHMatrix(foregroundTex);
     foregroundMatrix.preTranslate(SkIntToScalar(backgroundOffset.fX-foregroundOffset.fX),
                                   SkIntToScalar(backgroundOffset.fY-foregroundOffset.fY));
 
 
-    SkRect srcRect;
-    src.getBounds(&srcRect);
+    SkRect srcRect = SkRect::MakeWH(src.width(), src.height());
 
     GrPaint paint;
     paint.addColorTextureProcessor(foregroundTex, foregroundMatrix);
     paint.addColorProcessor(xferProcessor)->unref();
     context->drawRect(paint, SkMatrix::I(), srcRect);
 
     offset->fX = backgroundOffset.fX;
     offset->fY = backgroundOffset.fY;
-    WrapTexture(dst, src.width(), src.height(), result);
+    if (!WrapTexture(dst, src.width(), src.height(), result)) {
+        return false;
+    }
     return true;
 }
 
 #endif