Fix for degenerate stroke rect

src/gpu/batches/GrRectBatchFactory.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "GrRectBatchFactory.h"
 
 #include "GrAAStrokeRectBatch.h"
@@ skipping 15 matching lines @@
         devStrokeSize.setAbs(devStrokeSize);
     } else {
         devStrokeSize.set(SK_Scalar1, SK_Scalar1);
     }
 
     const SkScalar dx = devStrokeSize.fX;
     const SkScalar dy = devStrokeSize.fY;
     const SkScalar rx = SkScalarMul(dx, SK_ScalarHalf);
     const SkScalar ry = SkScalarMul(dy, SK_ScalarHalf);
 
-    SkScalar spare;
-    {
-        SkScalar w = devRect.width() - dx;
-        SkScalar h = devRect.height() - dy;
-        spare = SkTMin(w, h);
-    }
-
     SkRect devOutside(devRect);
     devOutside.outset(rx, ry);
 
     bool miterStroke = true;
     // For hairlines, make bevel and round joins appear the same as mitered ones.
     // small miter limit means right angles show bevel...
     if ((width > 0) && (stroke.getJoin() != SkPaint::kMiter_Join ||
                         stroke.getMiter() < SK_ScalarSqrt2)) {
         miterStroke = false;
     }
 
-    if (spare <= 0 && miterStroke) {
-        return CreateAAFill(color, viewMatrix, devOutside, devOutside);
-    }
-
     SkRect devInside(devRect);

[robertphillips, 2015/09/20 15:49:40]

It seems like we could get cracking with this method.

-    devInside.inset(rx, ry);
+    SkScalar insideRx = SkTMin(devRect.width() / 2.0f, rx);
+    SkScalar insideRy = SkTMin(devRect.height() / 2.0f, ry);
+    devInside.inset(insideRx, insideRy);
 
     SkRect devOutsideAssist(devRect);
 
     // For bevel-stroke, use 2 SkRect instances(devOutside and devOutsideAssist)
     // to draw the outer of the rect. Because there are 8 vertices on the outer
     // edge, while vertex number of inner edge is 4, the same as miter-stroke.
     if (!miterStroke) {
         devOutside.inset(0, ry);
         devOutsideAssist.outset(0, ry);
     }
@@ skipping 13 matching lines @@
     viewMatrix.mapRect(&devInside, rects[1]);
 
     if (devInside.isEmpty()) {
         return CreateAAFill(color, viewMatrix, devOutside, devOutside);
     }
 
     return GrAAStrokeRectBatch::Create(color, viewMatrix, devOutside, devOutside, devInside, true);
 }
 
 };