aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorGravatar tomhudson <tomhudson@google.com>2016-07-28 07:36:28 -0700
committerGravatar Commit bot <commit-bot@chromium.org>2016-07-28 07:36:28 -0700
commit92772222bf548acf937761eb5150d5bb8e4a7bc4 (patch)
treed05831ae5b1b242f79ceac4ae4fc6d952ee02a51
parent8796ff1fbcee33006b62c1df6f1573510d6e4e7f (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.cpp6
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;
}