diff options
author | 2018-05-30 22:47:46 -0400 | |
---|---|---|
committer | 2018-05-31 10:48:06 +0000 | |
commit | 53a0298de3de8f7b2f2299675b91ae293724d37d (patch) | |
tree | 631fa1c6554a398d32c6ab2eb6b003dc0e581e0c /src/effects/SkEmbossMask.cpp | |
parent | 8c8e10c7d9c5db8b8ffe34b8cb956ac6c8a0e6c0 (diff) |
GrTessellator: fix for ping-pong split fuzzer hang.
When support for out-of-range intersections was added in
3b5a3fa8b1c11d4bd4499b040311f4c3553ebf8c, it was intended to support
splitting edges that are almost flat, where merging with the top or
bottom vertex would cause visual artifacts. However, it triggers too
often for other, non-nearly-flat cases, causing simplify() to loop
infinitely.
The fix is to support out-of-range intersections only if they differ by
1/2 machine epsilon. This also generalizes the
out_of_range_and_collinear() check, so it was removed.
Bug: 838978
Change-Id: I238f2b90e4b7ad647ecf072427ee38726e549581
Reviewed-on: https://skia-review.googlesource.com/130458
Reviewed-by: Stephen White <senorblanco@chromium.org>
Diffstat (limited to 'src/effects/SkEmbossMask.cpp')
0 files changed, 0 insertions, 0 deletions