commit 7062d01a338d28e878c3c28469f84d2af1389796
author bungeman <bungeman@google.com> Fri Jun 03 17:43:24 2016 -0700
committer Commit bot <commit-bot@chromium.org> Fri Jun 03 17:43:24 2016 -0700
tree a4dd70945da3fd4752f029d009e3a12b6f486030
parent 862dae548a8ef40804bc9146d5c7b9f360efa960

More Unref GrXPFactory.

Not releasing the reference was caught by the Skia asan bot.
All remaining occurences of this pattern have been updated.

This fixes "Make GrClipMaskManager stateless and push GrPipelineBuilder construction downstack".

TBR=herb

Review-Url: https://codereview.chromium.org/2037243002

src/gpu/GrClipMaskManager.cpp

diff --git a/src/gpu/GrClipMaskManager.cpp b/src/gpu/GrClipMaskManager.cpp
index ff95eea..fd741b9 100644
--- a/src/gpu/GrClipMaskManager.cpp
+++ b/src/gpu/GrClipMaskManager.cpp
@@ -683,7 +683,7 @@
                     if (!clipPath.isEmpty()) {
                         if (canRenderDirectToStencil) {
                             GrPaint paint;
-                            paint.setXPFactory(GrDisableColorXPFactory::Create());
+                            SkSafeUnref(paint.setXPFactory(GrDisableColorXPFactory::Create()));
                             paint.setAntiAlias(element->isAA());
 
                             GrPathRenderer::DrawPathArgs args;
@@ -724,7 +724,7 @@
                                       viewMatrix, element->getRect(), element->isAA(), *pass);
                     } else {
                         GrPaint paint;
-                        paint.setXPFactory(GrDisableColorXPFactory::Create());
+                        SkSafeUnref(paint.setXPFactory(GrDisableColorXPFactory::Create()));
                         paint.setAntiAlias(element->isAA());
 
                         GrPathRenderer::DrawPathArgs args;

src/gpu/batches/GrDefaultPathRenderer.cpp

diff --git a/src/gpu/batches/GrDefaultPathRenderer.cpp b/src/gpu/batches/GrDefaultPathRenderer.cpp
index 9077d92..1bcfc2d 100644
--- a/src/gpu/batches/GrDefaultPathRenderer.cpp
+++ b/src/gpu/batches/GrDefaultPathRenderer.cpp
@@ -629,7 +629,7 @@
     SkASSERT(SkPath::kInverseWinding_FillType != args.fPath->getFillType());
 
     GrPaint paint;
-    paint.setXPFactory(GrDisableColorXPFactory::Create());
+    SkSafeUnref(paint.setXPFactory(GrDisableColorXPFactory::Create()));
     paint.setAntiAlias(args.fIsAA);
 
     this->internalDrawPath(args.fDrawContext,

src/gpu/batches/GrMSAAPathRenderer.cpp

diff --git a/src/gpu/batches/GrMSAAPathRenderer.cpp b/src/gpu/batches/GrMSAAPathRenderer.cpp
index 887c527..2983f58 100644
--- a/src/gpu/batches/GrMSAAPathRenderer.cpp
+++ b/src/gpu/batches/GrMSAAPathRenderer.cpp
@@ -751,7 +751,7 @@
     SkASSERT(SkPath::kInverseWinding_FillType != args.fPath->getFillType());
 
     GrPaint paint;
-    paint.setXPFactory(GrDisableColorXPFactory::Create());
+    SkSafeUnref(paint.setXPFactory(GrDisableColorXPFactory::Create()));
     paint.setAntiAlias(args.fIsAA);
 
     this->internalDrawPath(args.fDrawContext,

src/gpu/batches/GrStencilAndCoverPathRenderer.cpp

diff --git a/src/gpu/batches/GrStencilAndCoverPathRenderer.cpp b/src/gpu/batches/GrStencilAndCoverPathRenderer.cpp
index e43d05c..314e6c2 100644
--- a/src/gpu/batches/GrStencilAndCoverPathRenderer.cpp
+++ b/src/gpu/batches/GrStencilAndCoverPathRenderer.cpp
@@ -71,7 +71,7 @@
                               "GrStencilAndCoverPathRenderer::onStencilPath");
 
     GrPaint paint;
-    paint.setXPFactory(GrDisableColorXPFactory::Create());
+    SkSafeUnref(paint.setXPFactory(GrDisableColorXPFactory::Create()));
     paint.setAntiAlias(args.fIsAA);
 
     GrPipelineBuilder pipelineBuilder(paint, args.fDrawContext->isUnifiedMultisampled());