GrTessellator: abstract vertex allocation into caller

src/gpu/GrTessellator.h

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrTessellator_DEFINED
 #define GrTessellator_DEFINED
 
-#include "SkPath.h"
-#include "GrResourceProvider.h"
+#include "SkPoint.h"
+
+class SkPath;
+struct SkRect;
 
 /**
  * Provides utility functions for converting paths to a collection of triangles.
  */
 
 #define TESSELLATOR_WIREFRAME 0
 
 namespace GrTessellator {
 
+class VertexBuffer {

[bsalomon, 2016/03/10 15:46:24]

Reading that code below it isn't always totally obvious to me whether a vertex
buffer is a GrTessallator::VertexBuffer or a GrVertexBuffer. Can we use a
different name, e.g. VertexAllocator or VertexProvider?

[Stephen White, 2016/03/10 15:59:29]

Done.

+public:
+    virtual ~VertexBuffer() {}
+    virtual SkPoint* lock(int vertexCount) = 0;
+    virtual void unlock(int actualCount) = 0;
+};
+
 struct WindingVertex {
     SkPoint fPos;
     int fWinding;
 };
 
 // Triangulates a path to an array of vertices. Each triangle is represented as a set of three
 // WindingVertex entries, each of which contains the position and winding count (which is the same
 // for all three vertices of a triangle). The 'verts' out parameter is set to point to the resultant
 // vertex array. CALLER IS RESPONSIBLE for deleting this buffer to avoid a memory leak!
 int PathToVertices(const SkPath& path, SkScalar tolerance, const SkRect& clipBounds, 
                    WindingVertex** verts);
 
 int PathToTriangles(const SkPath& path, SkScalar tolerance, const SkRect& clipBounds, 
-                    GrResourceProvider* resourceProvider, 
-                    SkAutoTUnref<GrVertexBuffer>& vertexBuffer, bool canMapVB, bool* isLinear);
-
+                    VertexBuffer*, bool *isLinear);
 }
 
 #endif