Remove gpu-side clip mask merging from clip mask manager

src/gpu/GrClipMaskManager.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 "GrClipMaskManager.h"
 #include "GrCaps.h"
 #include "GrDrawContext.h"
@@ skipping 81 matching lines @@
         if (prOut) {
             *prOut = pr;
         }
         return SkToBool(!pr);
     }
 }
 
 // Determines whether it is possible to draw the element to both the stencil buffer and the
 // alpha mask simultaneously. If so and the element is a path a compatible path renderer is
 // also returned.
-static bool can_stencil_and_draw_element(GrContext* context,
+static GrPathRenderer* get_path_renderer(GrContext* context,
                                          GrPipelineBuilder* pipelineBuilder,
-                                         GrTexture* texture,
                                          const SkMatrix& viewMatrix,
-                                         const SkClipStack::Element* element,
-                                         GrPathRenderer** pr) {
-    pipelineBuilder->setRenderTarget(texture->asRenderTarget());
-
+                                         const SkClipStack::Element* element) {
+    GrPathRenderer* pr;
     static const bool kNeedsStencil = true;
-    return !path_needs_SW_renderer(context,
-                                   *pipelineBuilder,
-                                   viewMatrix,
-                                   element,
-                                   pr,
-                                   kNeedsStencil);
+    path_needs_SW_renderer(context, *pipelineBuilder, viewMatrix, element, &pr, kNeedsStencil);
+    return pr;
 }
 
 GrClipMaskManager::GrClipMaskManager(GrDrawTarget* drawTarget)
     : fDrawTarget(drawTarget)
     , fClipMode(kIgnoreClip_StencilClipMode) {
 }
 
-GrContext* GrClipMaskManager::getContext() { return fDrawTarget->cmmAccess().context(); }
+GrContext* GrClipMaskManager::getContext() {
+    return fDrawTarget->cmmAccess().context();
+}
 
+const GrCaps* GrClipMaskManager::caps() const {
+    return fDrawTarget->caps();
+}
+
+GrResourceProvider* GrClipMaskManager::resourceProvider() {
+    return fDrawTarget->cmmAccess().resourceProvider();
+}
 /*
  * This method traverses the clip stack to see if the GrSoftwarePathRenderer
  * will be used on any element. If so, it returns true to indicate that the
  * entire clip should be rendered in SW and then uploaded en masse to the gpu.
  */
 bool GrClipMaskManager::useSWOnlyPath(const GrPipelineBuilder& pipelineBuilder,
                                       const SkVector& clipToMaskOffset,
                                       const GrReducedClip::ElementList& elements) {
     // TODO: generalize this function so that when
     // a clip gets complex enough it can just be done in SW regardless
@@ skipping 339 matching lines @@
             args.fPath = &path;
             args.fStroke = &stroke;
             args.fAntiAlias = element->isAA();
             pr->drawPath(args);
             break;
         }
     }
     return true;
 }
 
-void GrClipMaskManager::mergeMask(GrPipelineBuilder* pipelineBuilder,
-                                  GrTexture* dstMask,
-                                  GrTexture* srcMask,
-                                  SkRegion::Op op,
-                                  const SkIRect& dstBound,
-                                  const SkIRect& srcBound) {
-    pipelineBuilder->setRenderTarget(dstMask->asRenderTarget());
-
-    // We want to invert the coverage here
-    set_coverage_drawing_xpf(op, false, pipelineBuilder);
-
-    SkMatrix sampleM;
-    sampleM.setIDiv(srcMask->width(), srcMask->height());
-
-    pipelineBuilder->addCoverageFragmentProcessor(
-        GrTextureDomainEffect::Create(srcMask,
-                                      sampleM,
-                                      GrTextureDomain::MakeTexelDomain(srcMask, srcBound),
-                                      GrTextureDomain::kDecal_Mode,
-                                      GrTextureParams::kNone_FilterMode))->unref();
-
-    // The color passed in here does not matter since the coverageSetOpXP won't read it.
-    fDrawTarget->drawNonAARect(*pipelineBuilder,
-                               GrColor_WHITE,
-                               SkMatrix::I(),
-                               SkRect::Make(dstBound));
-}
-
-GrTexture* GrClipMaskManager::createTempMask(int width, int height) {
-    GrSurfaceDesc desc;
-    desc.fFlags = kRenderTarget_GrSurfaceFlag;
-    desc.fWidth = width;
-    desc.fHeight = height;
-    if (this->getContext()->caps()->isConfigRenderable(kAlpha_8_GrPixelConfig, false)) {
-        desc.fConfig = kAlpha_8_GrPixelConfig;
-    } else {
-        desc.fConfig = kRGBA_8888_GrPixelConfig;
-    }
-
-    return this->getContext()->textureProvider()->createApproxTexture(desc);
-}
-
 ////////////////////////////////////////////////////////////////////////////////
 // Create a 8-bit clip mask in alpha
 
 static void GetClipMaskKey(int32_t clipGenID, const SkIRect& bounds, GrUniqueKey* key) {
     static const GrUniqueKey::Domain kDomain = GrUniqueKey::GenerateDomain();
     GrUniqueKey::Builder builder(key, kDomain, 3);
     builder[0] = clipGenID;
     builder[1] = SkToU16(bounds.fLeft) | (SkToU16(bounds.fRight) << 16);
     builder[2] = SkToU16(bounds.fTop) | (SkToU16(bounds.fBottom) << 16);
 }
 
 GrTexture* GrClipMaskManager::createCachedMask(int width, int height, const GrUniqueKey& key,
                                                bool renderTarget) {
     GrSurfaceDesc desc;
     desc.fWidth = width;
     desc.fHeight = height;
     desc.fFlags = renderTarget ? kRenderTarget_GrSurfaceFlag : kNone_GrSurfaceFlags;
-    if (!renderTarget || fDrawTarget->caps()->isConfigRenderable(kAlpha_8_GrPixelConfig, false)) {
+    if (!renderTarget || this->caps()->isConfigRenderable(kAlpha_8_GrPixelConfig, false)) {
         desc.fConfig = kAlpha_8_GrPixelConfig;
     } else {
         desc.fConfig = kRGBA_8888_GrPixelConfig;
     }
 
-    GrTexture* texture = fDrawTarget->cmmAccess().resourceProvider()->createApproxTexture(desc, 0);
+    GrTexture* texture = this->resourceProvider()->createApproxTexture(desc, 0);
     if (!texture) {
         return nullptr;
     }
     texture->resourcePriv().setUniqueKey(key);
     return texture;
 }
 
 GrTexture* GrClipMaskManager::createAlphaClipMask(int32_t elementsGenID,
                                                   GrReducedClip::InitialState initialState,
                                                   const GrReducedClip::ElementList& elements,
                                                   const SkVector& clipToMaskOffset,
                                                   const SkIRect& clipSpaceIBounds) {
-    GrResourceProvider* resourceProvider = fDrawTarget->cmmAccess().resourceProvider();
+    GrResourceProvider* resourceProvider = this->resourceProvider();
     GrUniqueKey key;
     GetClipMaskKey(elementsGenID, clipSpaceIBounds, &key);
     if (GrTexture* texture = resourceProvider->findAndRefTextureByUniqueKey(key)) {
         return texture;
     }
 
+    // There's no texture in the cache. Let's try to allocate it then.
     SkAutoTUnref<GrTexture> texture(this->createCachedMask(
         clipSpaceIBounds.width(), clipSpaceIBounds.height(), key, true));
-
-    // There's no texture in the cache. Let's try to allocate it then.
     if (!texture) {
         return nullptr;
     }
 
     // Set the matrix so that rendered clip elements are transformed to mask space from clip
     // space.
     const SkMatrix translate = SkMatrix::MakeTrans(clipToMaskOffset.fX, clipToMaskOffset.fY);
 
     // The texture may be larger than necessary, this rect represents the part of the texture
     // we populate with a rasterization of the clip.
     SkIRect maskSpaceIBounds = SkIRect::MakeWH(clipSpaceIBounds.width(), clipSpaceIBounds.height());
 
     // The scratch texture that we are drawing into can be substantially larger than the mask. Only
     // clear the part that we care about.
     fDrawTarget->clear(&maskSpaceIBounds,
                        GrReducedClip::kAllIn_InitialState == initialState ? 0xffffffff : 0x00000000,
                        true,
                        texture->asRenderTarget());
 
     // When we use the stencil in the below loop it is important to have this clip installed.
     // The second pass that zeros the stencil buffer renders the rect maskSpaceIBounds so the first
     // pass must not set values outside of this bounds or stencil values outside the rect won't be
     // cleared.
-    GrClip clip(maskSpaceIBounds);
-    SkAutoTUnref<GrTexture> temp;
+    const GrClip clip(maskSpaceIBounds);
 
     // walk through each clip element and perform its set op
     for (GrReducedClip::ElementList::Iter iter = elements.headIter(); iter.get(); iter.next()) {
         const Element* element = iter.get();
         SkRegion::Op op = element->getOp();
         bool invert = element->isInverseFilled();
         if (invert || SkRegion::kIntersect_Op == op || SkRegion::kReverseDifference_Op == op) {
             GrPipelineBuilder pipelineBuilder;
 
             pipelineBuilder.setClip(clip);
-            GrPathRenderer* pr = nullptr;
-            bool useTemp = !can_stencil_and_draw_element(this->getContext(), &pipelineBuilder,
-                                                         texture, translate, element, &pr);
+            pipelineBuilder.setRenderTarget(texture->asRenderTarget());
 
-            // useSWOnlyPath should now filter out all cases where gpu-side mask merging is
-            // performed. See skbug.com/4519 for rationale and details.
-            SkASSERT(!useTemp);
-
-            GrTexture* dst;
-            // This is the bounds of the clip element in the space of the alpha-mask. The temporary
-            // mask buffer can be substantially larger than the actually clip stack element. We
-            // touch the minimum number of pixels necessary and use decal mode to combine it with
-            // the accumulator.
-            SkIRect maskSpaceElementIBounds;
-
-            if (useTemp) {
-                if (invert) {
-                    maskSpaceElementIBounds = maskSpaceIBounds;
-                } else {
-                    SkRect elementBounds = element->getBounds();
-                    elementBounds.offset(clipToMaskOffset);
-                    elementBounds.roundOut(&maskSpaceElementIBounds);
-                }
-
-                if (!temp) {
-                    temp.reset(this->createTempMask(maskSpaceIBounds.fRight,
-                                                    maskSpaceIBounds.fBottom));
-                    if (!temp) {
-                        texture->resourcePriv().removeUniqueKey();
-                        return nullptr;
-                    }
-                }
-                dst = temp;
-                // clear the temp target and set blend to replace
-                fDrawTarget->clear(&maskSpaceElementIBounds,
-                                   invert ? 0xffffffff : 0x00000000,
-                                   true,
-                                   dst->asRenderTarget());
-                set_coverage_drawing_xpf(SkRegion::kReplace_Op, invert, &pipelineBuilder);
-            } else {
-                // draw directly into the result with the stencil set to make the pixels affected
-                // by the clip shape be non-zero.
-                dst = texture;
-                GR_STATIC_CONST_SAME_STENCIL(kStencilInElement,
-                                             kReplace_StencilOp,
-                                             kReplace_StencilOp,
-                                             kAlways_StencilFunc,
-                                             0xffff,
-                                             0xffff,
-                                             0xffff);
-                pipelineBuilder.setStencil(kStencilInElement);
-                set_coverage_drawing_xpf(op, invert, &pipelineBuilder);
+            GrPathRenderer* pr = get_path_renderer(this->getContext(), &pipelineBuilder,
+                                                   translate, element);
+            if (Element::kRect_Type != element->getType() && !pr) {
+                // useSWOnlyPath should now filter out all cases where gpu-side mask merging would
+                // be performed (i.e., pr would be NULL for a non-rect path). See skbug.com/4519
+                // for rationale and details.
+                SkASSERT(0);
+                continue;
             }
 
-            if (!this->drawElement(&pipelineBuilder, translate, dst, element, pr)) {
+            // draw directly into the result with the stencil set to make the pixels affected
+            // by the clip shape be non-zero.
+            GR_STATIC_CONST_SAME_STENCIL(kStencilInElement,
+                                         kReplace_StencilOp,
+                                         kReplace_StencilOp,
+                                         kAlways_StencilFunc,
+                                         0xffff,
+                                         0xffff,
+                                         0xffff);
+            pipelineBuilder.setStencil(kStencilInElement);
+            set_coverage_drawing_xpf(op, invert, &pipelineBuilder);
+
+            if (!this->drawElement(&pipelineBuilder, translate, texture, element, pr)) {
                 texture->resourcePriv().removeUniqueKey();
                 return nullptr;
             }
 
-            if (useTemp) {
+            {
                 GrPipelineBuilder backgroundPipelineBuilder;
                 backgroundPipelineBuilder.setRenderTarget(texture->asRenderTarget());
 
-                // Now draw into the accumulator using the real operation and the temp buffer as a
-                // texture
-                this->mergeMask(&backgroundPipelineBuilder,
-                                texture,
-                                temp,
-                                op,
-                                maskSpaceIBounds,
-                                maskSpaceElementIBounds);
-            } else {
-                GrPipelineBuilder backgroundPipelineBuilder;
-                backgroundPipelineBuilder.setRenderTarget(texture->asRenderTarget());
-
                 set_coverage_drawing_xpf(op, !invert, &backgroundPipelineBuilder);
                 // Draw to the exterior pixels (those with a zero stencil value).
                 GR_STATIC_CONST_SAME_STENCIL(kDrawOutsideElement,
                                              kZero_StencilOp,
                                              kZero_StencilOp,
                                              kEqual_StencilFunc,
                                              0xffff,
                                              0x0000,
                                              0xffff);
                 backgroundPipelineBuilder.setStencil(kDrawOutsideElement);
@@ skipping 19 matching lines @@
 // Create a 1-bit clip mask in the stencil buffer. 'devClipBounds' are in device
 // (as opposed to canvas) coordinates
 bool GrClipMaskManager::createStencilClipMask(GrRenderTarget* rt,
                                               int32_t elementsGenID,
                                               GrReducedClip::InitialState initialState,
                                               const GrReducedClip::ElementList& elements,
                                               const SkIRect& clipSpaceIBounds,
                                               const SkIPoint& clipSpaceToStencilOffset) {
     SkASSERT(rt);
 
-    GrStencilAttachment* stencilAttachment =
-        fDrawTarget->cmmAccess().resourceProvider()->attachStencilAttachment(rt);
+    GrStencilAttachment* stencilAttachment = this->resourceProvider()->attachStencilAttachment(rt);
     if (nullptr == stencilAttachment) {
         return false;
     }
 
     if (stencilAttachment->mustRenderClip(elementsGenID, clipSpaceIBounds, clipSpaceToStencilOffset)) {
         stencilAttachment->setLastClip(elementsGenID, clipSpaceIBounds, clipSpaceToStencilOffset);
         // Set the matrix so that rendered clip elements are transformed from clip to stencil space.
         SkVector translate = {
             SkIntToScalar(clipSpaceToStencilOffset.fX),
             SkIntToScalar(clipSpaceToStencilOffset.fY)
@@ skipping 235 matching lines @@
             settings = basic_apply_stencil_clip_settings();
         } else {
             return;
         }
     } else {
         settings = pipelineBuilder.getStencil();
     }
 
     int stencilBits = 0;
     GrRenderTarget* rt = pipelineBuilder.getRenderTarget();
-    GrStencilAttachment* stencilAttachment = 
-        fDrawTarget->cmmAccess().resourceProvider()->attachStencilAttachment(rt);
+    GrStencilAttachment* stencilAttachment = this->resourceProvider()->attachStencilAttachment(rt);
     if (stencilAttachment) {
         stencilBits = stencilAttachment->bits();
     }
 
-    SkASSERT(fDrawTarget->caps()->stencilWrapOpsSupport() || !settings.usesWrapOp());
-    SkASSERT(fDrawTarget->caps()->twoSidedStencilSupport() || !settings.isTwoSided());
+    SkASSERT(this->caps()->stencilWrapOpsSupport() || !settings.usesWrapOp());
+    SkASSERT(this->caps()->twoSidedStencilSupport() || !settings.isTwoSided());
     this->adjustStencilParams(&settings, fClipMode, stencilBits);
     ars->set(&pipelineBuilder);
     ars->setStencil(settings);
 }
 
 void GrClipMaskManager::adjustStencilParams(GrStencilSettings* settings,
                                             StencilClipMode mode,
                                             int stencilBitCnt) {
     SkASSERT(stencilBitCnt > 0);
 
     if (kModifyClip_StencilClipMode == mode) {
         // We assume that this clip manager itself is drawing to the GrGpu and
         // has already setup the correct values.
         return;
     }
 
     unsigned int clipBit = (1 << (stencilBitCnt - 1));
     unsigned int userBits = clipBit - 1;
 
     GrStencilSettings::Face face = GrStencilSettings::kFront_Face;
-    bool twoSided = fDrawTarget->caps()->twoSidedStencilSupport();
+    bool twoSided = this->caps()->twoSidedStencilSupport();
 
     bool finished = false;
     while (!finished) {
         GrStencilFunc func = settings->func(face);
         uint16_t writeMask = settings->writeMask(face);
         uint16_t funcMask = settings->funcMask(face);
         uint16_t funcRef = settings->funcRef(face);
 
         SkASSERT((unsigned) func < kStencilFuncCount);
 
@@ skipping 51 matching lines @@
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 GrTexture* GrClipMaskManager::createSoftwareClipMask(int32_t elementsGenID,
                                                      GrReducedClip::InitialState initialState,
                                                      const GrReducedClip::ElementList& elements,
                                                      const SkVector& clipToMaskOffset,
                                                      const SkIRect& clipSpaceIBounds) {
     GrUniqueKey key;
     GetClipMaskKey(elementsGenID, clipSpaceIBounds, &key);
-    GrResourceProvider* resourceProvider = fDrawTarget->cmmAccess().resourceProvider();
+    GrResourceProvider* resourceProvider = this->resourceProvider();
     if (GrTexture* texture = resourceProvider->findAndRefTextureByUniqueKey(key)) {
         return texture;
     }
 
     // The mask texture may be larger than necessary. We round out the clip space bounds and pin
     // the top left corner of the resulting rect to the top left of the texture.
     SkIRect maskSpaceIBounds = SkIRect::MakeWH(clipSpaceIBounds.width(), clipSpaceIBounds.height());
 
     GrSWMaskHelper helper(this->getContext());
 
@@ skipping 51 matching lines @@
 
 ////////////////////////////////////////////////////////////////////////////////
 
 void GrClipMaskManager::adjustPathStencilParams(const GrStencilAttachment* stencilAttachment,
                                                 GrStencilSettings* settings) {
     if (stencilAttachment) {
         int stencilBits = stencilAttachment->bits();
         this->adjustStencilParams(settings, fClipMode, stencilBits);
     }
 }