Trivial cleanup of copying GrPipelineBuilder
authorjoshualitt <joshualitt@chromium.org>
Thu, 9 Jul 2015 19:17:44 +0000 (12:17 -0700)
committerCommit bot <commit-bot@chromium.org>
Thu, 9 Jul 2015 19:17:44 +0000 (12:17 -0700)
TBR=robertphillips@google.com
BUG=skia:

Review URL: https://codereview.chromium.org/1229983003

src/gpu/GrClipMaskManager.cpp
src/gpu/GrSoftwarePathRenderer.cpp

index 76cfbf6..fa9c30e 100644 (file)
@@ -842,24 +842,23 @@ bool GrClipMaskManager::createStencilClipMask(GrRenderTarget* rt,
             // element directly or a bounding rect of the entire clip.
             fClipMode = kModifyClip_StencilClipMode;
             for (int p = 0; p < passes; ++p) {
-                GrPipelineBuilder pipelineBuilderCopy(pipelineBuilder);
-                *pipelineBuilderCopy.stencil() = stencilSettings[p];
+                *pipelineBuilder.stencil() = stencilSettings[p];
 
                 if (canDrawDirectToClip) {
                     if (Element::kRect_Type == element->getType()) {
                         // We need this AGP until everything is in GrBatch
-                        fClipTarget->drawSimpleRect(&pipelineBuilderCopy,
+                        fClipTarget->drawSimpleRect(&pipelineBuilder,
                                                     GrColor_WHITE,
                                                     viewMatrix,
                                                     element->getRect());
                     } else {
-                        pr->drawPath(fClipTarget, &pipelineBuilderCopy, GrColor_WHITE,
+                        pr->drawPath(fClipTarget, &pipelineBuilder, GrColor_WHITE,
                                      viewMatrix, clipPath, stroke, false);
                     }
                 } else {
                     // The view matrix is setup to do clip space -> stencil space translation, so
                     // draw rect in clip space.
-                    fClipTarget->drawSimpleRect(&pipelineBuilderCopy,
+                    fClipTarget->drawSimpleRect(&pipelineBuilder,
                                                 GrColor_WHITE,
                                                 viewMatrix,
                                                 SkRect::Make(clipSpaceIBounds));
index d9b58a8..3d5bc3e 100644 (file)
@@ -146,8 +146,7 @@ bool GrSoftwarePathRenderer::onDrawPath(GrDrawTarget* target,
         return false;
     }
 
-    GrPipelineBuilder copy = *pipelineBuilder;
-    GrSWMaskHelper::DrawToTargetWithPathMask(texture, target, &copy, color, viewMatrix,
+    GrSWMaskHelper::DrawToTargetWithPathMask(texture, target, pipelineBuilder, color, viewMatrix,
                                              devPathBounds);
 
     if (path.isInverseFillType()) {