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

Skip bilerp for integral-translate-only matrices

Browse Source 
BUG=chromium:744674

Change-Id: I053a0c26b0f644faa11f469b8a6486302a5efae6
Reviewed-on: https://skia-review.googlesource.com/24126
Commit-Queue: Florin Malita <fmalita@chromium.org>
Reviewed-by: Mike Reed <reed@google.com>
This commit is contained in:
Florin Malita 2017-07-17 16:26:57 -04:00 committed by Skia Commit-Bot
parent 3cf6b79e50
commit 0983268dd5
1 changed files with 14 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/core/SkBitmapProcState.cpp
View File

@ -61,6 +61,18 @@ static bool just_trans_general(const SkMatrix& matrix) {
&& SkScalarNearlyZero(matrix[SkMatrix::kMScaleY] - SK_Scalar1, tol);
}
/**
* Determine if the matrix can be treated as integral-only-translate,
* for the purpose of filtering.
*/
static bool just_trans_integral(const SkMatrix& m) {
static constexpr SkScalar tol = SK_Scalar1 / 256;
return m.getType() <= SkMatrix::kTranslate_Mask
&& SkScalarNearlyEqual(m.getTranslateX(), SkScalarRoundToScalar(m.getTranslateX()), tol)
&& SkScalarNearlyEqual(m.getTranslateY(), SkScalarRoundToScalar(m.getTranslateY()), tol);
}
static bool valid_for_filtering(unsigned dimension) {
// for filtering, width and height must fit in 14bits, since we use steal
// 2 bits from each to store our 4bit subpixel data
@ -118,12 +130,11 @@ bool SkBitmapProcInfo::init(const SkMatrix& inv, const SkPaint& paint) {
fInvType = fInvMatrix.getType();
if (kLow_SkFilterQuality == fFilterQuality &&
!valid_for_filtering(fPixmap.width() | fPixmap.height())) {
(!valid_for_filtering(fPixmap.width() | fPixmap.height()) ||
just_trans_integral(fInvMatrix))) {
fFilterQuality = kNone_SkFilterQuality;
}
// TODO: skip bilerp for integral translates
return true;
}