diff options
author | tomhudson <tomhudson@google.com> | 2016-07-28 07:36:28 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-07-28 07:36:28 -0700 |
commit | 92772222bf548acf937761eb5150d5bb8e4a7bc4 (patch) | |
tree | d05831ae5b1b242f79ceac4ae4fc6d952ee02a51 | |
parent | 8796ff1fbcee33006b62c1df6f1573510d6e4e7f (diff) |
Revert of align cap hairline first point (patchset #2 id:20001 of https://codereview.chromium.org/2189583003/ )
Reason for revert:
Speculative revert in case this is blocking our roll.
Original issue's description:
> align cap hairline first point
>
> Hairlines with square caps move the endpoints out by 1/2 pixel.
> The last point needs to be the first adjusted point, instead
> of the point supplied to move to.
>
> R=fmalita@chromium.org
> TBR=reed@google.com
> BUG=617658
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2189583003
>
> Committed: https://skia.googlesource.com/skia/+/52f85d4d09b815fce6bfe296a1dd2cbf8d792f05
TBR=fmalita@chromium.org,reed@google.com,caryclark@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=617658
Review-Url: https://codereview.chromium.org/2188483007
-rw-r--r-- | src/core/SkScan_Hairline.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/core/SkScan_Hairline.cpp b/src/core/SkScan_Hairline.cpp index 9be2b8fd12..083dc0df5f 100644 --- a/src/core/SkScan_Hairline.cpp +++ b/src/core/SkScan_Hairline.cpp @@ -538,12 +538,10 @@ void hair_path(const SkPath& path, const SkRasterClip& rclip, SkBlitter* blitter if (SkPaint::kButt_Cap != capStyle) { prevVerb = SkPath::kDone_Verb; } - bool firstPtSet = false; while ((verb = iter.next(pts)) != SkPath::kDone_Verb) { switch (verb) { case SkPath::kMove_Verb: firstPt = lastPt = pts[0]; - firstPtSet = false; break; case SkPath::kLine_Verb: if (SkPaint::kButt_Cap != capStyle) { @@ -594,10 +592,6 @@ void hair_path(const SkPath& path, const SkRasterClip& rclip, SkBlitter* blitter case SkPath::kDone_Verb: break; } - if (!firstPtSet && SkPath::kLine_Verb <= verb && verb <= SkPath::kCubic_Verb) { - firstPt = lastPt = pts[0]; - firstPtSet = true; - } if (SkPaint::kButt_Cap != capStyle) { prevVerb = verb; } |