diff options
| author |  Yuqian Li <liyuqian@google.com> | 2018-03-09 14:23:50 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-03-09 19:48:46 +0000 |
| commit | 629503de26b9685dfb3a184add3f0967cee8caa3 (patch) | |
| tree | a65ae5183ccd93e2fbd7bf51d84e08ecceaec06f /src/core/SkCoverageDelta.h | |
| parent | 749efb6ee2b48dad07548742d83c98e468663714 (diff) | |
Do not set record to empty if it's already computed
Otherwise, the following test will fail:
./out/Debug/dm --config t8888 -m strokes_round
Bug: skia:
Change-Id: Ifc618fc42c6f387eae7b6aadd8424001c3e1464e
Reviewed-on: https://skia-review.googlesource.com/113440
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Yuqian Li <liyuqian@google.com>
Diffstat (limited to 'src/core/SkCoverageDelta.h')
| -rw-r--r-- | src/core/SkCoverageDelta.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/core/SkCoverageDelta.h b/src/core/SkCoverageDelta.h index e0a50f4f3b..58c83ce343 100644 --- a/src/core/SkCoverageDelta.h +++ b/src/core/SkCoverageDelta.h @@ -203,7 +203,9 @@ struct SkDAARecord { void setEmpty() { fType = Type::kEmpty; } static inline void SetEmpty(SkDAARecord* record) { // record may be nullptr #ifdef SK_DEBUG - if (record) { + // If type != kToBeComputed, then we're in the draw phase and we shouldn't set it to empty + // because being empty in one tile does not imply emptiness in other tiles. + if (record && record->fType == Type::kToBeComputed) { record->setEmpty(); } #endif |