Revert "Revert "Make GrTextureOp disable coverage AA when rect falls on integers.""
This reverts commit 36be574e7b
.
Change-Id: Ifdc06cb617ce5c736f861a715bb6f34791364e57
Reviewed-on: https://skia-review.googlesource.com/130507
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
This commit is contained in:
parent
18dbfd05e4
commit
594b64c485
@ -709,15 +709,28 @@ __attribute__((no_sanitize("float-cast-overflow")))
|
||||
, fFinalized(0)
|
||||
, fAllowSRGBInputs(allowSRGBInputs ? 1 : 0) {
|
||||
SkASSERT(aaType != GrAAType::kMixedSamples);
|
||||
|
||||
const Draw& draw = fDraws.emplace_back(srcRect, 0, GrPerspQuad(dstRect, viewMatrix),
|
||||
constraint, color);
|
||||
fPerspective = viewMatrix.hasPerspective();
|
||||
fDomain = (bool)draw.domain();
|
||||
SkRect bounds;
|
||||
bounds = draw.quad().bounds();
|
||||
auto quad = GrPerspQuad(dstRect, viewMatrix);
|
||||
auto bounds = quad.bounds();
|
||||
#ifndef SK_DONT_DROP_UNNECESSARY_AA_IN_TEXTURE_OP
|
||||
if (GrAAType::kCoverage == this->aaType() && viewMatrix.rectStaysRect()) {
|
||||
// Disable coverage AA when rect falls on integers in device space.
|
||||
auto is_int = [](float f) { return f == sk_float_floor(f); };
|
||||
if (is_int(bounds.fLeft) && is_int(bounds.fTop) && is_int(bounds.fRight) &&
|
||||
is_int(bounds.fBottom)) {
|
||||
fAAType = static_cast<unsigned>(GrAAType::kNone);
|
||||
// We may have had a strict constraint with nearest filter soley due to possible AA
|
||||
// bloat. In that case it's no longer necessary.
|
||||
if (constraint == SkCanvas::kStrict_SrcRectConstraint &&
|
||||
filter == GrSamplerState::Filter::kNearest) {
|
||||
constraint = SkCanvas::kFast_SrcRectConstraint;
|
||||
}
|
||||
}
|
||||
}
|
||||
#endif
|
||||
const auto& draw = fDraws.emplace_back(srcRect, 0, quad, constraint, color);
|
||||
this->setBounds(bounds, HasAABloat::kNo, IsZeroArea::kNo);
|
||||
|
||||
fDomain = static_cast<bool>(draw.domain());
|
||||
fMaxApproxDstPixelArea = RectSizeAsSizeT(bounds);
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user