Commit ef3b2187 authored Nov 06, 2017 by Derek Sollenberger

If the absolute value of a sweep angle >= 360 then draw it as an oval.

Skia's drawArc API is defined to keep winding when drawing an arc
with a sweep angle > 360.  This allows for special path effects
and for points on the arc to potentially be double drawn. This is
not the case for the HWUI implementation that optimistically turns
the call into a drawOval.  This CL updates Skia's pipeline to be
in line with the HWUI implemenation.

Test: CtsGraphicsTestCases were added to cover this
Bug: 68764873
Change-Id: I22cd15a53cd2df44421518256d45a2b5d0525a86

parent 622597fb

libs/hwui/SkiaCanvas.cpp

+5 −1

Original line number	Diff line number	Diff line
		@@ -521,8 +521,12 @@ void SkiaCanvas::drawArc(float left, float top, float right, float bottom, float
		float sweepAngle, bool useCenter, const SkPaint& paint) {
		if (CC_UNLIKELY(paint.nothingToDraw())) return;
		SkRect arc = SkRect::MakeLTRB(left, top, right, bottom);
		if (fabs(sweepAngle) >= 360.0f) {
		mCanvas->drawOval(arc, paint);
		} else {
		mCanvas->drawArc(arc, startAngle, sweepAngle, useCenter, paint);
		}
		}

		void SkiaCanvas::drawPath(const SkPath& path, const SkPaint& paint) {
		if (CC_UNLIKELY(paint.nothingToDraw())) return;