Fix SkPath::arcTo when sweepAngle is tiny and radius is big

gm/addarc.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 "gm.h"
 #include "SkAnimTimer.h"
 #include "SkCanvas.h"
@@ skipping 201 matching lines @@
 private:
     SkScalar fRotate;
 
     typedef skiagm::GM INHERITED;
 };
 DEF_GM( return new FillCircleGM; )
 
 //////////////////////
 
 static void html_canvas_arc(SkPath* path, SkScalar x, SkScalar y, SkScalar r, SkScalar start,
-                            SkScalar end, bool ccw) {
+                            SkScalar end, bool ccw, bool callArcTo) {
     SkRect bounds = { x - r, y - r, x + r, y + r };
     SkScalar sweep = ccw ? end - start : start - end;
-    path->arcTo(bounds, start, sweep, false);
+    if (callArcTo)
+        path->arcTo(bounds, start, sweep, false);
+    else
+        path->addArc(bounds, start, sweep);
 }
 
 // Lifted from canvas-arc-circumference-fill-diffs.html
 class ManyArcsGM : public skiagm::GM {
 public:
     ManyArcsGM() {}
 
 protected:
     SkString onShortName() override { return SkString("manyarcs"); }
 
@@ skipping 26 matching lines @@
             if (i == SK_ARRAY_COUNT(startAngles)) {
                 anticlockwise = true;
                 sign = -1;
             }
             SkScalar startAngle = startAngles[i % SK_ARRAY_COUNT(startAngles)] * sign;
             canvas->save();
             for (size_t j = 0; j < SK_ARRAY_COUNT(sweepAngles); ++j) {
                 SkPath path;
                 path.moveTo(0, 2);
                 html_canvas_arc(&path, 18, 15, 10, startAngle, startAngle + (sweepAngles[j] * sign),
-                                anticlockwise);
+                                anticlockwise, true);
                 path.lineTo(0, 28);
                 canvas->drawPath(path, paint);
                 canvas->translate(30, 0);
             }
             canvas->restore();
             canvas->translate(0, 40);
         }
     }
 
 private:
     typedef skiagm::GM INHERITED;
 };
 DEF_GM( return new ManyArcsGM; )
+
+// Lifted from https://bugs.chromium.org/p/chromium/issues/detail?id=640031
+class TinyAngleBigRadiusArcsGM : public skiagm::GM {
+public:
+    TinyAngleBigRadiusArcsGM() {}
+
+protected:
+    SkString onShortName() override { return SkString("tinyanglearcs"); }
+
+    SkISize onISize() override { return SkISize::Make(620, 330); }
+
+    void onDraw(SkCanvas* canvas) override {
+        SkPaint paint;
+        paint.setAntiAlias(true);
+        paint.setStyle(SkPaint::kStroke_Style);
+
+        canvas->translate(50, 50);
+
+        SkPath path;
+        path.moveTo(50, 20);
+        path.lineTo(50, 0);
+        // A combination of tiny sweepAngle + large radius, we should draw a line.
+        html_canvas_arc(&path, 50, 100000, 100000, 270, 270.0f - 0.00572957795f,
+                        false, true);
+        path.lineTo(60, 20);
+        html_canvas_arc(&path, 50, 100000, 99980, 270.0f - 0.00572957795f, 270,
+                        false, false);
+        canvas->drawPath(path, paint);
+    }
+
+private:
+    typedef skiagm::GM INHERITED;
+};
+DEF_GM( return new TinyAngleBigRadiusArcsGM; )