Revert of Minor cleanup in GrTessellatingPathRenderer.

Revert of Minor cleanup in GrTessellatingPathRenderer. (patchset #1 id:1 of https://codereview.chromium.org/1322023002/ )

Reason for revert:
Patch is incorrect -- floating point errors can cause zero-area monotone polys to be skipped, resulting in a smaller vertex count than estimated.

Reproduce as follows: 

out/Debug/SampleApp --slide Fuzzer --msaa 4

switch to GPU mode. Result:

../../src/gpu/batches/GrTessellatingPathRenderer.cpp:1500: failed assertion "static_cast<int>(end - verts) == vertexCount"

Original issue's description:
> Minor cleanup in GrTessellatingPathRenderer.
> 
> Vertex counts are always exact, so don't bother handling the case
> where they're different. Just assert.
> Rename variables to reflect.
> 
> BUG=skia:
> 
> Committed: https://skia.googlesource.com/skia/+/9389b871c3b0f06bb34626cf9bdbfe0c93779327

TBR=bsalomon@google.com,senorblanco@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Committed: https://skia.googlesource.com/skia/+/06f989a61e4b0237e338a54d2cc0efd531cf0041

[Stephen White, 2015-09-02 15:50:41]

Created Revert of Minor cleanup in GrTessellatingPathRenderer.

[commit-bot: I haz the power, 2015-09-02 15:50:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1321723006/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1321723006/1

[commit-bot: I haz the power, 2015-09-02 15:50:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-02 15:50:54]

Failed to apply patch for src/gpu/GrTessellatingPathRenderer.cpp:
While running git apply --index -3 -p1;
  error: src/gpu/GrTessellatingPathRenderer.cpp: does not exist in index

Patch:       src/gpu/GrTessellatingPathRenderer.cpp
Index: src/gpu/GrTessellatingPathRenderer.cpp
diff --git a/src/gpu/GrTessellatingPathRenderer.cpp
b/src/gpu/GrTessellatingPathRenderer.cpp
index
bc9a6b5d040b362a96b6a86451ed1227a813c93c..46fa280cfd86c6e67a429c547a1e987e1af6f135
100644
--- a/src/gpu/GrTessellatingPathRenderer.cpp
+++ b/src/gpu/GrTessellatingPathRenderer.cpp
@@ -1337,10 +1337,10 @@
 
 struct TessInfo {
     SkScalar  fTolerance;
-    int       fVertexCount;
+    int       fCount;
 };
 
-bool cache_match(GrVertexBuffer* vertexBuffer, SkScalar tol, int* vertexCount)
{
+bool cache_match(GrVertexBuffer* vertexBuffer, SkScalar tol, int* actualCount)
{
     if (!vertexBuffer) {
         return false;
     }
@@ -1348,7 +1348,7 @@
     SkASSERT(data);
     const TessInfo* info = static_cast<const TessInfo*>(data->data());
     if (info->fTolerance == 0 || info->fTolerance < 3.0f * tol) {
-        *vertexCount = info->fVertexCount;
+        *actualCount = info->fCount;
         return true;
     }
     return false;
@@ -1471,17 +1471,17 @@
         path_to_contours(path, tol, fClipBounds, contours.get(), alloc,
&isLinear);
         Poly* polys;
         polys = contours_to_polys(contours.get(), contourCnt, c, alloc);
-        int vertexCount = 0;
+        int count = 0;
         for (Poly* poly = polys; poly; poly = poly->fNext) {
             if (apply_fill_type(fillType, poly->fWinding) && poly->fCount >= 3)
{
-                vertexCount += (poly->fCount - 2) * (WIREFRAME ? 6 : 3);
-            }
-        }
-        if (0 == vertexCount) {
+                count += (poly->fCount - 2) * (WIREFRAME ? 6 : 3);
+            }
+        }
+        if (0 == count) {
             return 0;
         }
 
-        size_t size = vertexCount * sizeof(SkPoint);
+        size_t size = count * sizeof(SkPoint);
         if (!vertexBuffer.get() || vertexBuffer->gpuMemorySize() < size) {
             vertexBuffer.reset(resourceProvider->createVertexBuffer(
                 size, GrResourceProvider::kStatic_BufferUsage, 0));
@@ -1494,15 +1494,16 @@
         if (canMapVB) {
             verts = static_cast<SkPoint*>(vertexBuffer->map());
         } else {
-            verts = new SkPoint[vertexCount];
-        }
-        SkDEBUGCODE(SkPoint* end = ) polys_to_triangles(polys, fillType,
verts);
-        SkASSERT(static_cast<int>(end - verts) == vertexCount);
-        LOG("vertex count: %d\n", vertexCount);
+            verts = new SkPoint[count];
+        }
+        SkPoint* end = polys_to_triangles(polys, fillType, verts);
+        int actualCount = static_cast<int>(end - verts);
+        LOG("actual count: %d\n", actualCount);
+        SkASSERT(actualCount <= count);
         if (canMapVB) {
             vertexBuffer->unmap();
         } else {
-            vertexBuffer->updateData(verts, vertexCount * sizeof(SkPoint));
+            vertexBuffer->updateData(verts, actualCount * sizeof(SkPoint));
             delete[] verts;
         }
 
@@ -1510,13 +1511,13 @@
         if (!fPath.isVolatile()) {
             TessInfo info;
             info.fTolerance = isLinear ? 0 : tol;
-            info.fVertexCount = vertexCount;
+            info.fCount = actualCount;
             SkAutoTUnref<SkData> data(SkData::NewWithCopy(&info,
sizeof(info)));
             key->setCustomData(data.get());
             resourceProvider->assignUniqueKeyToResource(*key,
vertexBuffer.get());
             SkPathPriv::AddGenIDChangeListener(fPath, new
PathInvalidator(*key));
         }
-        return vertexCount;
+        return actualCount;
     }
 
     void onPrepareDraws(Target* target) override {
@@ -1537,16 +1538,16 @@
         builder.finish();
         GrResourceProvider* rp = target->resourceProvider();
         SkAutoTUnref<GrVertexBuffer>
vertexBuffer(rp->findAndRefTByUniqueKey<GrVertexBuffer>(key));
-        int vertexCount;
+        int actualCount;
         SkScalar screenSpaceTol = GrPathUtils::kDefaultTolerance;
         SkScalar tol = GrPathUtils::scaleToleranceToSrc(
             screenSpaceTol, fViewMatrix, fPath.getBounds());
-        if (!cache_match(vertexBuffer.get(), tol, &vertexCount)) {
+        if (!cache_match(vertexBuffer.get(), tol, &actualCount)) {
             bool canMapVB = GrCaps::kNone_MapFlags !=
target->caps().mapBufferFlags();
-            vertexCount = tessellate(&key, rp, vertexBuffer, canMapVB);
-        }
-
-        if (vertexCount == 0) {
+            actualCount = tessellate(&key, rp, vertexBuffer, canMapVB);
+        }
+
+        if (actualCount == 0) {
             return;
         }
 
@@ -1575,7 +1576,7 @@
         GrPrimitiveType primitiveType = WIREFRAME ? kLines_GrPrimitiveType
                                                   : kTriangles_GrPrimitiveType;
         GrVertices vertices;
-        vertices.init(primitiveType, vertexBuffer.get(), 0, vertexCount);
+        vertices.init(primitiveType, vertexBuffer.get(), 0, actualCount);
         target->draw(vertices);
     }

[Stephen White, 2015-09-02 16:05:11]

The CQ bit was checked by senorblanco@chromium.org

[commit-bot: I haz the power, 2015-09-02 16:05:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1321723006/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1321723006/40001

[commit-bot: I haz the power, 2015-09-02 16:05:20]

Committed patchset #2 (id:40001) as
https://skia.googlesource.com/skia/+/06f989a61e4b0237e338a54d2cc0efd531cf0041