Implement instanced rendering for simple shapes

src/gpu/GrDrawContext.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "GrBatchTest.h"
 #include "GrColor.h"
 #include "GrDrawContext.h"
 #include "GrDrawContextPriv.h"
 #include "GrDrawingManager.h"
 #include "GrOvalRenderer.h"
 #include "GrPathRenderer.h"
 #include "GrRenderTarget.h"
 #include "GrRenderTargetPriv.h"
 #include "GrResourceProvider.h"
+#include "GrInstancedRendering.h"
 #include "SkSurfacePriv.h"
 
 #include "batches/GrBatch.h"
 #include "batches/GrDrawAtlasBatch.h"
 #include "batches/GrDrawVerticesBatch.h"
 #include "batches/GrRectBatchFactory.h"
 #include "batches/GrNinePatch.h" // TODO Factory
 
 #include "effects/GrRRectEffect.h"
 
@@ skipping 36 matching lines @@
     : fDrawingManager(drawingMgr)
     , fRenderTarget(rt)
     , fDrawTarget(SkSafeRef(rt->getLastDrawTarget()))
     , fContext(context)
     , fSurfaceProps(SkSurfacePropsCopyOrDefault(surfaceProps))
     , fAuditTrail(auditTrail)
 #ifdef SK_DEBUG
     , fSingleOwner(singleOwner)
 #endif
 {
+    fInstancedRenderingFlags = GrInstancedRendering::kColorWrite_Flag;
+    if (fRenderTarget->isStencilBufferMultisampled()) {
+        fInstancedRenderingFlags |= GrInstancedRendering::kStencilBufferMSAA_Flag;
+    }
+    if (fRenderTarget->isUnifiedMultisampled()) {
+        fInstancedRenderingFlags |= GrInstancedRendering::kColorBufferMSAA_Flag;
+    }
     SkDEBUGCODE(this->validate();)
 }
 
 #ifdef SK_DEBUG
 void GrDrawContext::validate() const {
     SkASSERT(fRenderTarget);
     ASSERT_OWNED_RESOURCE(fRenderTarget);
 
     if (fDrawTarget && !fDrawTarget->isClosed()) {
         SkASSERT(fRenderTarget->getLastDrawTarget() == fDrawTarget);
@@ skipping 158 matching lines @@
 
 static inline bool rect_contains_inclusive(const SkRect& rect, const SkPoint& point) {
     return point.fX >= rect.fLeft && point.fX <= rect.fRight &&
            point.fY >= rect.fTop && point.fY <= rect.fBottom;
 }
 
 static bool view_matrix_ok_for_aa_fill_rect(const SkMatrix& viewMatrix) {
     return viewMatrix.preservesRightAngles();
 }
 
-static bool should_apply_coverage_aa(const GrPaint& paint, GrRenderTarget* rt) {
-    return paint.isAntiAlias() && !rt->isUnifiedMultisampled();
+static bool should_apply_coverage_aa(const GrPaint& paint, GrRenderTarget* rt, bool* useHWAA) {
+    if (!paint.isAntiAlias()) {
+        *useHWAA = false;
+        return false;
+    } else {
+        *useHWAA = rt->isUnifiedMultisampled();
+        return !rt->isUnifiedMultisampled();
+    }
 }
 
 GrDrawBatch* GrDrawContext::getFillRectBatch(const GrPaint& paint,
                                              const SkMatrix& viewMatrix,
-                                             const SkRect& rect) {
-
-    GrDrawBatch* batch = nullptr;
-    if (should_apply_coverage_aa(paint, fRenderTarget)) {
+                                             const SkRect& rect,
+                                             uint32_t extraInstRenderFlags,
+                                             bool* useHWAA) {
+    if (GrInstancedRendering* ir = this->getDrawTarget()->instancedRendering()) {
+        if (GrDrawBatch* batch = ir->recordRect(rect, viewMatrix, paint.getColor(),
+                                                paint.isAntiAlias(),
+                                                fInstancedRenderingFlags | extraInstRenderFlags,
+                                                useHWAA)) {
+            return batch;
+        }
+    }
+    if (should_apply_coverage_aa(paint, fRenderTarget, useHWAA)) {
         // The fill path can handle rotation but not skew.
         if (view_matrix_ok_for_aa_fill_rect(viewMatrix)) {
             SkRect devBoundRect;
             viewMatrix.mapRect(&devBoundRect, rect);
-            batch = GrRectBatchFactory::CreateAAFill(paint.getColor(), viewMatrix,
-                                                     rect, devBoundRect);
+            return GrRectBatchFactory::CreateAAFill(paint.getColor(), viewMatrix, rect,
+                                                    devBoundRect);
         }
     } else {
         // filled BW rect
-        batch = GrRectBatchFactory::CreateNonAAFill(paint.getColor(), viewMatrix, rect,
-                                                    nullptr, nullptr);
+        return GrRectBatchFactory::CreateNonAAFill(paint.getColor(), viewMatrix, rect, nullptr,
+                                                   nullptr);
     }
 
-    return batch;
+    return nullptr;
 }
 
 void GrDrawContext::drawRect(const GrClip& clip,
                              const GrPaint& paint,
                              const SkMatrix& viewMatrix,
                              const SkRect& rect,
                              const GrStrokeInfo* strokeInfo) {
     ASSERT_SINGLE_OWNER
     RETURN_IF_ABANDONED
     SkDEBUGCODE(this->validate();)
@@ skipping 34 matching lines @@
                 GrColor clearColor;
                 if (paint.isConstantBlendedColor(&clearColor)) {
                     this->getDrawTarget()->clear(nullptr, clearColor, true, fRenderTarget);
                     return;
                 }
             }
         }
     }
 
     bool snapToPixelCenters = false;
+    bool useHWAA;
     SkAutoTUnref<GrDrawBatch> batch;
     if (width < 0) {
-        batch.reset(this->getFillRectBatch(paint, viewMatrix, rect));
+        batch.reset(this->getFillRectBatch(paint, viewMatrix, rect, 0, &useHWAA));
     } else {
         GrColor color = paint.getColor();
 
-        if (should_apply_coverage_aa(paint, fRenderTarget)) {
+        if (should_apply_coverage_aa(paint, fRenderTarget, &useHWAA)) {
             // The stroke path needs the rect to remain axis aligned (no rotation or skew).
             if (viewMatrix.rectStaysRect()) {
                 batch.reset(GrRectBatchFactory::CreateAAStroke(color, viewMatrix, rect,
                                                                *strokeInfo));
             }
         } else {
             // Non-AA hairlines are snapped to pixel centers to make which pixels are hit
             // deterministic
             snapToPixelCenters = (0 == width && !fRenderTarget->isUnifiedMultisampled());
             batch.reset(GrRectBatchFactory::CreateNonAAStroke(color, viewMatrix, rect,
                                                               width, snapToPixelCenters));
 
             // Depending on sub-pixel coordinates and the particular GPU, we may lose a corner of
             // hairline rects. We jam all the vertices to pixel centers to avoid this, but not
             // when MSAA is enabled because it can cause ugly artifacts.
         }
     }
 
     if (batch) {
         GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
+        pipelineBuilder.setState(GrPipelineBuilder::kHWAntialias_Flag, useHWAA);
 
         if (snapToPixelCenters) {
             pipelineBuilder.setState(GrPipelineBuilder::kSnapVerticesToPixelCenters_Flag,
                                      snapToPixelCenters);
         }
 
         this->getDrawTarget()->drawBatch(pipelineBuilder, batch);
         return;
     }
 
@@ skipping 15 matching lines @@
     RETURN_FALSE_IF_ABANDONED_PRIV
     SkDEBUGCODE(fDrawContext->validate();)
     GR_AUDIT_TRAIL_AUTO_FRAME(fDrawContext->fAuditTrail, "GrDrawContext::stencilRect");
 
     AutoCheckFlush acf(fDrawContext->fDrawingManager);
 
     GrPaint paint;
     paint.setAntiAlias(doAA);
     paint.setCoverageSetOpXPFactory(op, invert);
 
-    SkAutoTUnref<GrDrawBatch> batch(fDrawContext->getFillRectBatch(paint, viewMatrix, rect));
+    bool useHWAA;
+    SkAutoTUnref<GrDrawBatch> batch(
+        fDrawContext->getFillRectBatch(paint, viewMatrix, rect,
+                                       GrInstancedRendering::kStencilWrite_Flag, &useHWAA));
     if (batch) {
         GrPipelineBuilder pipelineBuilder(paint, fDrawContext->fRenderTarget, GrClip::WideOpen());
+        pipelineBuilder.setState(GrPipelineBuilder::kHWAntialias_Flag, useHWAA);
         pipelineBuilder.setStencil(ss);
 
         fDrawContext->getDrawTarget()->drawBatch(pipelineBuilder, batch, scissorRect);
         return true;
     }
 
     SkPath path;
     path.setIsVolatile(true);
     path.addRect(rect);
     return this->drawAndStencilPath(scissorRect, ss, op, invert, doAA, viewMatrix, path);
 }
 
 void GrDrawContext::fillRectToRect(const GrClip& clip,
                                    const GrPaint& paint,
                                    const SkMatrix& viewMatrix,
                                    const SkRect& rectToDraw,
                                    const SkRect& localRect) {
     ASSERT_SINGLE_OWNER
     RETURN_IF_ABANDONED
     SkDEBUGCODE(this->validate();)
     GR_AUDIT_TRAIL_AUTO_FRAME(fAuditTrail, "GrDrawContext::fillRectToRect");
 
     AutoCheckFlush acf(fDrawingManager);
 
+    bool useHWAA;
     SkAutoTUnref<GrDrawBatch> batch;
-    if (should_apply_coverage_aa(paint, fRenderTarget) &&
-        view_matrix_ok_for_aa_fill_rect(viewMatrix)) {
-        batch.reset(GrAAFillRectBatch::CreateWithLocalRect(paint.getColor(), viewMatrix, rectToDraw,
-                                                           localRect));
-    } else {
-        batch.reset(GrRectBatchFactory::CreateNonAAFill(paint.getColor(), viewMatrix, rectToDraw,
-                                                        &localRect, nullptr));
+    if (GrInstancedRendering* ir = this->getDrawTarget()->instancedRendering()) {
+        batch.reset(ir->recordRect(rectToDraw, viewMatrix, paint.getColor(), localRect,
+                                   paint.isAntiAlias(), fInstancedRenderingFlags, &useHWAA));
+    }
+    if (!batch) {
+        if (should_apply_coverage_aa(paint, fRenderTarget, &useHWAA) &&
+            view_matrix_ok_for_aa_fill_rect(viewMatrix)) {
+            batch.reset(GrAAFillRectBatch::CreateWithLocalRect(paint.getColor(), viewMatrix,
+                                                               rectToDraw, localRect));
+        } else {
+            batch.reset(GrRectBatchFactory::CreateNonAAFill(paint.getColor(), viewMatrix,
+                                                            rectToDraw, &localRect, nullptr));
+        }
     }
 
-    if (batch) {
-        GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
-        this->drawBatch(&pipelineBuilder, batch);
-    }
+    GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
+    pipelineBuilder.setState(GrPipelineBuilder::kHWAntialias_Flag, useHWAA);
+    this->drawBatch(&pipelineBuilder, batch);
 }
 
 void GrDrawContext::fillRectWithLocalMatrix(const GrClip& clip,
                                             const GrPaint& paint,
                                             const SkMatrix& viewMatrix,
                                             const SkRect& rectToDraw,
                                             const SkMatrix& localMatrix) {
     ASSERT_SINGLE_OWNER
     RETURN_IF_ABANDONED
     SkDEBUGCODE(this->validate();)
     GR_AUDIT_TRAIL_AUTO_FRAME(fAuditTrail, "GrDrawContext::fillRectWithLocalMatrix");
 
     AutoCheckFlush acf(fDrawingManager);
 
+    bool useHWAA;
     SkAutoTUnref<GrDrawBatch> batch;
-    if (should_apply_coverage_aa(paint, fRenderTarget) &&
-        view_matrix_ok_for_aa_fill_rect(viewMatrix)) {
-        batch.reset(GrAAFillRectBatch::Create(paint.getColor(), viewMatrix, localMatrix,
-                                              rectToDraw));
-    } else {
-        batch.reset(GrRectBatchFactory::CreateNonAAFill(paint.getColor(), viewMatrix, rectToDraw,
-                                                        nullptr, &localMatrix));
+    if (GrInstancedRendering* ir = this->getDrawTarget()->instancedRendering()) {
+        batch.reset(ir->recordRect(rectToDraw, viewMatrix, paint.getColor(), localMatrix,
+                                   paint.isAntiAlias(), fInstancedRenderingFlags, &useHWAA));
+    }
+    if (!batch) {
+        if (should_apply_coverage_aa(paint, fRenderTarget, &useHWAA) &&
+            view_matrix_ok_for_aa_fill_rect(viewMatrix)) {
+            batch.reset(GrAAFillRectBatch::Create(paint.getColor(), viewMatrix, localMatrix,
+                                                  rectToDraw));
+        } else {
+            batch.reset(GrRectBatchFactory::CreateNonAAFill(paint.getColor(), viewMatrix,
+                                                            rectToDraw, nullptr, &localMatrix));
+        }
     }
 
     GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
+    pipelineBuilder.setState(GrPipelineBuilder::kHWAntialias_Flag, useHWAA);
     this->getDrawTarget()->drawBatch(pipelineBuilder, batch);
 }
 
 void GrDrawContext::drawVertices(const GrClip& clip,
                                  const GrPaint& paint,
                                  const SkMatrix& viewMatrix,
                                  GrPrimitiveType primitiveType,
                                  int vertexCount,
                                  const SkPoint positions[],
                                  const SkPoint texCoords[],
@@ skipping 73 matching lines @@
     GR_AUDIT_TRAIL_AUTO_FRAME(fAuditTrail, "GrDrawContext::drawRRect");
 
     if (rrect.isEmpty()) {
        return;
     }
 
     SkASSERT(!strokeInfo.isDashed()); // this should've been devolved to a path in SkGpuDevice
 
     AutoCheckFlush acf(fDrawingManager);
 
-    if (should_apply_coverage_aa(paint, fRenderTarget)) {
-        GrShaderCaps* shaderCaps = fContext->caps()->shaderCaps();
-
-        SkAutoTUnref<GrDrawBatch> batch(GrOvalRenderer::CreateRRectBatch(paint.getColor(),
-                                                                         viewMatrix,
-                                                                         rrect,
-                                                                         strokeInfo,
-                                                                         shaderCaps));
-        if (batch) {
-            GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
-            this->getDrawTarget()->drawBatch(pipelineBuilder, batch);
-            return;
-        }
+    bool useHWAA;
+    SkAutoTUnref<GrDrawBatch> batch;
+    if (this->getDrawTarget()->instancedRendering() && strokeInfo.isFillStyle()) {
+        GrInstancedRendering* ir = this->getDrawTarget()->instancedRendering();
+        batch.reset(ir->recordRRect(rrect, viewMatrix, paint.getColor(), paint.isAntiAlias(),
+                                    fInstancedRenderingFlags, &useHWAA));
+    }
+    if (!batch && should_apply_coverage_aa(paint, fRenderTarget, &useHWAA)) {
+        batch.reset(GrOvalRenderer::CreateRRectBatch(paint.getColor(), viewMatrix, rrect,
+                                                     strokeInfo, fContext->caps()->shaderCaps()));
     }
 
-    SkPath path;
-    path.setIsVolatile(true);
-    path.addRRect(rrect);
-    this->internalDrawPath(clip, paint, viewMatrix, path, strokeInfo);
+    if (batch) {
+        GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
+        pipelineBuilder.setState(GrPipelineBuilder::kHWAntialias_Flag, useHWAA);
+        this->drawBatch(&pipelineBuilder, batch);
+    } else {
+        SkPath path;
+        path.setIsVolatile(true);
+        path.addRRect(rrect);
+        this->internalDrawPath(clip, paint, viewMatrix, path, strokeInfo);
+    }
 }
 
 bool GrDrawContext::drawFilledDRRect(const GrClip& clip,
                                      const GrPaint& paintIn,
                                      const SkMatrix& viewMatrix,
                                      const SkRRect& origOuter,
                                      const SkRRect& origInner) {
     SkASSERT(!origInner.isEmpty());
     SkASSERT(!origOuter.isEmpty());
 
+    if (GrInstancedRendering* ir = this->getDrawTarget()->instancedRendering()) {
+        bool useHWAA;
+        SkAutoTUnref<GrDrawBatch> batch(ir->recordDRRect(origOuter, origInner, viewMatrix,
+                                                         paintIn.getColor(), paintIn.isAntiAlias(),
+                                                         fInstancedRenderingFlags, &useHWAA));
+        if (batch) {
+            GrPipelineBuilder pipelineBuilder(paintIn, fRenderTarget, clip);
+            pipelineBuilder.setState(GrPipelineBuilder::kHWAntialias_Flag, useHWAA);
+            this->drawBatch(&pipelineBuilder, batch);
+            return true;
+        }
+    }
+
     bool applyAA = paintIn.isAntiAlias() && !fRenderTarget->isUnifiedMultisampled();
 
     GrPrimitiveEdgeType innerEdgeType = applyAA ? kInverseFillAA_GrProcessorEdgeType :
                                                   kInverseFillBW_GrProcessorEdgeType;
     GrPrimitiveEdgeType outerEdgeType = applyAA ? kFillAA_GrProcessorEdgeType :
                                                   kFillBW_GrProcessorEdgeType;
 
     SkTCopyOnFirstWrite<SkRRect> inner(origInner), outer(origOuter);
     SkMatrix inverseVM;
     if (!viewMatrix.isIdentity()) {
@@ skipping 78 matching lines @@
     GR_AUDIT_TRAIL_AUTO_FRAME(fAuditTrail, "GrDrawContext::drawOval");
 
     if (oval.isEmpty()) {
        return;
     }
 
     SkASSERT(!strokeInfo.isDashed()); // this should've been devolved to a path in SkGpuDevice
 
     AutoCheckFlush acf(fDrawingManager);
 
-    if (should_apply_coverage_aa(paint, fRenderTarget)) {
-        GrShaderCaps* shaderCaps = fContext->caps()->shaderCaps();
-        SkAutoTUnref<GrDrawBatch> batch(GrOvalRenderer::CreateOvalBatch(paint.getColor(),
-                                                                        viewMatrix,
-                                                                        oval,
-                                                                        strokeInfo,
-                                                                        shaderCaps));
-        if (batch) {
-            GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
-            this->getDrawTarget()->drawBatch(pipelineBuilder, batch);
-            return;
-        }
+    bool useHWAA;
+    SkAutoTUnref<GrDrawBatch> batch;
+    if (this->getDrawTarget()->instancedRendering() && strokeInfo.isFillStyle()) {
+        GrInstancedRendering* ir = this->getDrawTarget()->instancedRendering();
+        batch.reset(ir->recordOval(oval, viewMatrix, paint.getColor(), paint.isAntiAlias(),
+                                   fInstancedRenderingFlags, &useHWAA));
+    }
+    if (!batch && should_apply_coverage_aa(paint, fRenderTarget, &useHWAA)) {
+        batch.reset(GrOvalRenderer::CreateOvalBatch(paint.getColor(), viewMatrix, oval, strokeInfo,
+                                                    fContext->caps()->shaderCaps()));
     }
 
-    SkPath path;
-    path.setIsVolatile(true);
-    path.addOval(oval);
-    this->internalDrawPath(clip, paint, viewMatrix, path, strokeInfo);
+    if (batch) {
+        GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
+        pipelineBuilder.setState(GrPipelineBuilder::kHWAntialias_Flag, useHWAA);
+        this->drawBatch(&pipelineBuilder, batch);
+    } else {
+        SkPath path;
+        path.setIsVolatile(true);
+        path.addOval(oval);
+        this->internalDrawPath(clip, paint, viewMatrix, path, strokeInfo);
+    }
 }
 
 void GrDrawContext::drawImageNine(const GrClip& clip,
                                   const GrPaint& paint,
                                   const SkMatrix& viewMatrix,
                                   int imageWidth,
                                   int imageHeight,
                                   const SkIRect& center,
                                   const SkRect& dst) {
     ASSERT_SINGLE_OWNER
@@ skipping 98 matching lines @@
     GR_AUDIT_TRAIL_AUTO_FRAME(fAuditTrail, "GrDrawContext::drawPath");
 
     if (path.isEmpty()) {
        if (path.isInverseFillType()) {
            this->drawPaint(clip, paint, viewMatrix);
        }
        return;
     }
 
     AutoCheckFlush acf(fDrawingManager);
+    SkAutoTUnref<GrDrawBatch> fastBatch;
+    bool useHWAA;
 
-    if (should_apply_coverage_aa(paint, fRenderTarget) && !strokeInfo.isDashed()) {
-        if (strokeInfo.getWidth() < 0 && !path.isConvex()) {
-            // Concave AA paths are expensive - try to avoid them for special cases
-            SkRect rects[2];
-
-            if (is_nested_rects(viewMatrix, path, strokeInfo, rects)) {
-                SkAutoTUnref<GrDrawBatch> batch(GrRectBatchFactory::CreateAAFillNestedRects(
-                    paint.getColor(), viewMatrix, rects));
-
-                GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
-                this->getDrawTarget()->drawBatch(pipelineBuilder, batch);
-                return;
-            }
+    // Paths are expensive - try to avoid them for special cases
+    SkRect nestedRects[2];
+    SkRect ovalRect;
+    if (strokeInfo.isFillStyle() && !path.isConvex() &&
+        is_nested_rects(viewMatrix, path, strokeInfo, nestedRects)) {
+        if (GrInstancedRendering* ir = this->getDrawTarget()->instancedRendering()) {
+            SkASSERT(nestedRects[0].contains(nestedRects[1]));
+            fastBatch.reset(ir->recordDRRect(SkRRect::MakeRect(nestedRects[0]),
+                                             SkRRect::MakeRect(nestedRects[1]), viewMatrix,
+                                             paint.getColor(), paint.isAntiAlias(),
+                                             fInstancedRenderingFlags, &useHWAA));
         }
-        SkRect ovalRect;
-        bool isOval = path.isOval(&ovalRect);
-
-        if (isOval && !path.isInverseFillType()) {
-            GrShaderCaps* shaderCaps = fContext->caps()->shaderCaps();
-            SkAutoTUnref<GrDrawBatch> batch(GrOvalRenderer::CreateOvalBatch(paint.getColor(),
-                                                                            viewMatrix,
-                                                                            ovalRect,
-                                                                            strokeInfo,
-                                                                            shaderCaps));
-            if (batch) {
-                GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
-                this->getDrawTarget()->drawBatch(pipelineBuilder, batch);
-                return;
-            }
+        if (!fastBatch && should_apply_coverage_aa(paint, fRenderTarget, &useHWAA)) {
+            fastBatch.reset(GrRectBatchFactory::CreateAAFillNestedRects(paint.getColor(),
+                                                                        viewMatrix, nestedRects));
+        }
+    } else if (!path.isInverseFillType() && path.isOval(&ovalRect)) {
+        if (this->getDrawTarget()->instancedRendering() && strokeInfo.isFillStyle()) {
+            GrInstancedRendering* ir = this->getDrawTarget()->instancedRendering();
+            fastBatch.reset(ir->recordOval(ovalRect, viewMatrix, paint.getColor(),
+                                           paint.isAntiAlias(), fInstancedRenderingFlags,
+                                           &useHWAA));
+        }
+        if (!fastBatch && !strokeInfo.isDashed() &&
+            should_apply_coverage_aa(paint, fRenderTarget, &useHWAA)) {
+            fastBatch.reset(GrOvalRenderer::CreateOvalBatch(paint.getColor(), viewMatrix,
+                                                            ovalRect, strokeInfo,
+                                                            fContext->caps()->shaderCaps()));
         }
     }
 
-    // Note that internalDrawPath may sw-rasterize the path into a scratch texture.
-    // Scratch textures can be recycled after they are returned to the texture
-    // cache. This presents a potential hazard for buffered drawing. However,
-    // the writePixels that uploads to the scratch will perform a flush so we're
-    // OK.
-    this->internalDrawPath(clip, paint, viewMatrix, path, strokeInfo);
+    if (fastBatch) {
+        GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
+        pipelineBuilder.setState(GrPipelineBuilder::kHWAntialias_Flag, useHWAA);
+        this->drawBatch(&pipelineBuilder, fastBatch);
+    } else {
+        // Note that internalDrawPath may sw-rasterize the path into a scratch texture.
+        // Scratch textures can be recycled after they are returned to the texture
+        // cache. This presents a potential hazard for buffered drawing. However,
+        // the writePixels that uploads to the scratch will perform a flush so we're
+        // OK.
+        this->internalDrawPath(clip, paint, viewMatrix, path, strokeInfo);
+    }
 }
 
 bool GrDrawContextPriv::drawAndStencilPath(const SkIRect* scissorRect,
                                            const GrStencilSettings& ss,
                                            SkRegion::Op op,
                                            bool invert,
                                            bool doAA,
                                            const SkMatrix& viewMatrix,
                                            const SkPath& path) {
     ASSERT_SINGLE_OWNER_PRIV
@@ skipping 70 matching lines @@
                                      const SkPath& path,
                                      const GrStrokeInfo& strokeInfo) {
     ASSERT_SINGLE_OWNER
     RETURN_IF_ABANDONED
     SkASSERT(!path.isEmpty());
 
     // An Assumption here is that path renderer would use some form of tweaking
     // the src color (either the input alpha or in the frag shader) to implement
     // aa. If we have some future driver-mojo path AA that can do the right
     // thing WRT to the blend then we'll need some query on the PR.
-    bool useCoverageAA = should_apply_coverage_aa(paint, fRenderTarget);
+    bool useHWAA;
+    bool useCoverageAA = should_apply_coverage_aa(paint, fRenderTarget, &useHWAA);
     const bool isStencilDisabled = true;
     bool isStencilBufferMSAA = fRenderTarget->isStencilBufferMultisampled();
 
     const GrPathRendererChain::DrawType type =
         useCoverageAA ? GrPathRendererChain::kColorAntiAlias_DrawType
                       : GrPathRendererChain::kColor_DrawType;
 
     const SkPath* pathPtr = &path;
     SkTLazy<SkPath> tmpPath;
     const GrStrokeInfo* strokeInfoPtr = &strokeInfo;
@@ skipping 55 matching lines @@
     }
 
     if (nullptr == pr) {
 #ifdef SK_DEBUG
         SkDebugf("Unable to find path renderer compatible with path.\n");
 #endif
         return;
     }
 
     GrPipelineBuilder pipelineBuilder(paint, fRenderTarget, clip);
+    pipelineBuilder.setState(GrPipelineBuilder::kHWAntialias_Flag, useHWAA);
 
     GrPathRenderer::DrawPathArgs args;
     args.fTarget = this->getDrawTarget();
     args.fResourceProvider = fDrawingManager->getContext()->resourceProvider();
     args.fPipelineBuilder = &pipelineBuilder;
     args.fColor = paint.getColor();
     args.fViewMatrix = &viewMatrix;
     args.fPath = pathPtr;
     args.fStroke = strokeInfoPtr;
     args.fAntiAlias = useCoverageAA;
     args.fGammaCorrect = this->isGammaCorrect();
     pr->drawPath(args);
 }
 
 void GrDrawContext::drawBatch(GrPipelineBuilder* pipelineBuilder, GrDrawBatch* batch) {
     ASSERT_SINGLE_OWNER
     RETURN_IF_ABANDONED
     SkDEBUGCODE(this->validate();)
     GR_AUDIT_TRAIL_AUTO_FRAME(fAuditTrail, "GrDrawContext::drawBatch");
 
     this->getDrawTarget()->drawBatch(*pipelineBuilder, batch);
 }