diff options
| author | Stephen White <senorblanco@chromium.org> | 2018-09-26 17:25:59 -0400 |
|---|---|---|
| committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-09-26 23:57:59 +0000 |
| commit | cfe1264d7465b0791073969ba6375dcca20afec9 (patch) | |
| tree | fce5c0d0339599a2db87eb2d8c77a0eaa155d2f5 /gn | |
| parent | 5ea41fc89b264f2cab7058bf48429f87761f65e9 (diff) | |
| download | skqp-cfe1264d7465b0791073969ba6375dcca20afec9.tar.gz | |
GrTessellator: two fixes for boundary simplification artifacts.
The "pointy vertex" test was only considering the distance from the
next (outgoing) edge to the previous point. It must also consider the
distance from the previous (incoming) edge to the next point. If
either are less than a quarter pixel, the vertex is considered pointy
and should be removed. (884166)
Also (887103), when an interior region was completely removed due to
boundary simplification, it would leave a degenerate edge consisting
of the same vertex. So avoid introducing a join edge when prev == next.
Bug: 884166, 887103
Change-Id: I7f1d5b98e418d8f2a1c11643259d3cd74d08f286
Reviewed-on: https://skia-review.googlesource.com/157220
Commit-Queue: Stephen White <senorblanco@chromium.org>
Reviewed-by: Robert Phillips <robertphillips@google.com>
Diffstat (limited to 'gn')
| -rw-r--r-- | gn/gm.gni | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -97,6 +97,8 @@ gm_sources = [ "$_gm/crbug_691386.cpp", "$_gm/crbug_788500.cpp", "$_gm/crbug_847759.cpp", + "$_gm/crbug_884166.cpp", + "$_gm/crbug_887103.cpp", "$_gm/croppedrects.cpp", "$_gm/crosscontextimage.cpp", "$_gm/cubicpaths.cpp", |