Remove GrPipelineBuilder from getPathRenderer call

include/gpu/GrContext.h

 /*
  * Copyright 2010 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrContext_DEFINED
 #define GrContext_DEFINED
 
@@ skipping 315 matching lines @@
     GrLayerCache* getLayerCache() { return fLayerCache.get(); }
     GrTextBlobCache* getTextBlobCache() { return fTextBlobCache; }
     bool abandoned() const;
     GrResourceProvider* resourceProvider() { return fResourceProvider; }
     const GrResourceProvider* resourceProvider() const { return fResourceProvider; }
     GrResourceCache* getResourceCache() { return fResourceCache; }
 
     // Called by tests that draw directly to the context via GrDrawTarget
     void getTestTarget(GrTestTarget*);
 
     GrPathRenderer* getPathRenderer(

[bsalomon, 2015/10/27 19:54:03]

Maybe we should just pass the RT instead of the isMSAA thing?

Should we use a args struct here?

Is there a more semantic understanding of the stencilIsDisabled param, e.g.
callerWillStencil? I don't remember exactly why we need this.

[robertphillips, 2015/10/29 14:51:31]

I'm on the fence about this one. Passing the RT is clearer but the RT is
overkill for the usage.
 
Done.
I think that will have to be a different CL. I've quickly looked at the code and
have come away un-enlightened (there both a kIsDisabled flag and a kNotDisabled
flag).

-                    const GrPipelineBuilder&,
                     const SkMatrix& viewMatrix,
                     const SkPath& path,
                     const GrStrokeInfo& stroke,
                     bool allowSW,
-                    GrPathRendererChain::DrawType drawType = GrPathRendererChain::kColor_DrawType,
+                    GrPathRendererChain::DrawType drawType,
+                    bool stencilIsDisabled,
+                    bool stencilBufferIsMSAA,
                     GrPathRendererChain::StencilSupport* stencilSupport = NULL);
 
     /** Prints cache stats to the string if GR_CACHE_STATS == 1. */
     void dumpCacheStats(SkString*) const;
     void printCacheStats() const;
 
     /** Prints GPU stats to the string if GR_GPU_STATS == 1. */
     void dumpGpuStats(SkString*) const;
     void printGpuStats() const;
 
@@ skipping 87 matching lines @@
     /**
      * A callback similar to the above for use by the TextBlobCache
      * TODO move textblob draw calls below context so we can use the call above.
      */
     static void TextBlobCacheOverBudgetCB(void* data);
 
     typedef SkRefCnt INHERITED;
 };
 
 #endif