diff options
| author |  Cary Clark <caryclark@skia.org> | 2018-04-12 10:29:46 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-04-12 15:05:40 +0000 |
| commit | 31608c02c2533810587f5807c0c34f8ac19290b9 (patch) | |
| tree | c65f44e3cbd4fd9b900c8ef134ac39591637a7cb /src/core | |
| parent | ba375a88843160e6884023e9108ea84de8eb3a0f (diff) | |
another rect is path fix
This addresses comment #17 of skbug.com/7792.
The bug overshoots the end and exploits that the
first point tracked by close isn't the first
point in the rectangle.
Fixing this slightly regresses the example
in comment #14; before it was treated as a filled
rect but now it is not; this conservative approach
doesn't cause any other regressions.
bug7792 in pathfill.cpp verifies that all paths
in the bug draw correctly by comparing CPU and GPU.
R=robertphillips@google.com
Bug: 824145,skia:7792
Change-Id: I55bea023d2ad7456c8c3ebd9d1df95fe34e0a0d4
Reviewed-on: https://skia-review.googlesource.com/120996
Commit-Queue: Cary Clark <caryclark@skia.org>
Reviewed-by: Robert Phillips <robertphillips@google.com>
Diffstat (limited to 'src/core')
| -rw-r--r-- | src/core/SkPath.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/core/SkPath.cpp b/src/core/SkPath.cpp index 9e242376c0..942b95da4d 100644 --- a/src/core/SkPath.cpp +++ b/src/core/SkPath.cpp @@ -467,7 +467,7 @@ bool SkPath::isRectContour(bool allowPartial, int* currVerb, const SkPoint** pts switch (verb) { case kClose_Verb: savePts = pts; - pts = *ptsPtr; + pts = firstPt; autoClose = true; insertClose = false; accumulatingRect = false; |