AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Always turn off domain in GrTextureOp for no-aa/nearest

Browse Source 
Fixes an issue on Mali-400 where fragment shader domain check causes
precision loss of texture coords even when vertex texture coords are
inside the domain.

Change-Id: I3e1ea2c7ee7a73a0c1e8df76421afa432f986b88
Reviewed-on: https://skia-review.googlesource.com/c/159324
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
This commit is contained in:
Brian Salomon 2018-10-03 15:59:04 -04:00 committed by Skia Commit-Bot
parent e749c2f30c
commit f09abc58ad
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/gpu/ops/GrTextureOp.cpp
View File

@ -696,15 +696,17 @@ private:
if (GrAAType::kCoverage == this->aaType()) { if (GrAAType::kCoverage == this->aaType()) {
fAAType = static_cast<unsigned>(GrAAType::kNone); fAAType = static_cast<unsigned>(GrAAType::kNone);
aaFlags = GrQuadAAFlags::kNone; aaFlags = GrQuadAAFlags::kNone;
// We may have had a strict constraint with nearest filter solely due to
// possible AA bloat. In that case it's no longer necessary.
if (constraint == SkCanvas::kStrict_SrcRectConstraint &&
fFilter == GrSamplerState::Filter::kNearest) {
constraint = SkCanvas::kFast_SrcRectConstraint;
}
} }
} }
} }
// We may have had a strict constraint with nearest filter solely due to possible AA bloat.
// If we don't have (or determined we don't need) coverage AA then we can skip using a
// domain.
if (constraint == SkCanvas::kStrict_SrcRectConstraint &&
fFilter == GrSamplerState::Filter::kNearest &&
this->aaType() != GrAAType::kCoverage) {
constraint = SkCanvas::kFast_SrcRectConstraint;
}
const auto& draw = fDraws.emplace_back(srcRect, quad, aaFlags, constraint, color); const auto& draw = fDraws.emplace_back(srcRect, quad, aaFlags, constraint, color);
this->setBounds(bounds, HasAABloat::kNo, IsZeroArea::kNo); this->setBounds(bounds, HasAABloat::kNo, IsZeroArea::kNo);
fDomain = static_cast<bool>(draw.domain()); fDomain = static_cast<bool>(draw.domain());