Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(180)

Issue 2477373002: Combine analytic edges with tolerance (Closed)

Created:
4 years, 1 month ago by liyuqian
Modified:
4 years, 1 month ago
Reviewers:
f(malita), caryclark, reed1
CC:
reviews_skia.org
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Combine analytic edges with tolerance If not, we sometimes would end up with only one edge for a convex path. That either triggers SkASSERT(count >= 2) failure in debug build or SIGSEGV in release build. After the change, we should return 0 edges for such a path because everything is totally combined. Note that this change also makes the SkAnalyticEdge's CombineVertical function behave more similarly to SkEdge's CombineVertical function: SkEdge only compares fFirstY and fLastY which are integer values, which is equivalent to setting our tolerance to SK_Fixed1 (our current tolerance is 0x100, 1/256 of SK_Fixed1). And this is intentional. BUG=chromium:662914 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2477373002 Committed: https://skia.googlesource.com/skia/+/451ceba3544fa1ff810378f1e9a47c7d20f48a9f

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -4 lines) Patch
M gm/aaa.cpp View 1 chunk +11 lines, -0 lines 0 comments Download
M src/core/SkEdgeBuilder.cpp View 3 chunks +8 lines, -4 lines 0 comments Download

Messages

Total messages: 16 (12 generated)
liyuqian
This CL should fix: https://bugs.chromium.org/p/chromium/issues/detail?id=662914
4 years, 1 month ago (2016-11-07 15:36:05 UTC) #6
caryclark
lgtm
4 years, 1 month ago (2016-11-07 16:09:21 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2477373002/1
4 years, 1 month ago (2016-11-07 16:09:52 UTC) #13
commit-bot: I haz the power
4 years, 1 month ago (2016-11-07 16:10:47 UTC) #15
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/451ceba3544fa1ff810378f1e9a47c7d20f48a9f

Powered by Google App Engine
This is Rietveld 408576698