Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix failing TextureViewTest"

Browse Source
This commit is contained in:
John Reck
2018-11-27 16:39:47 +00:00
committed by Android (Google) Code Review
parent 6e7ff180a5 0aff62d12f
commit 82fec85ccb
3 changed files with 26 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
libs/hwui/Matrix.h
View File

@@ -27,6 +27,7 @@ namespace android {
namespace uirenderer {
#define SK_MATRIX_STRING "[%.2f %.2f %.2f] [%.2f %.2f %.2f] [%.2f %.2f %.2f]"
#define SK_MATRIX_STRING_V "[%.9f %.9f %.9f] [%.9f %.9f %.9f] [%.9f %.9f %.9f]"
#define SK_MATRIX_ARGS(m) \
(m)->get(0), (m)->get(1), (m)->get(2), (m)->get(3), (m)->get(4), (m)->get(5), (m)->get(6), \
(m)->get(7), (m)->get(8)

26
libs/hwui/pipeline/skia/LayerDrawable.cpp
View File

@@ -14,6 +14,7 @@
* limitations under the License.
*/
#include <utils/MathUtils.h>
#include "LayerDrawable.h"
#include "GrBackendSurface.h"
@@ -32,6 +33,24 @@ void LayerDrawable::onDraw(SkCanvas* canvas) {
}
}
// This is a less-strict matrix.isTranslate() that will still report being translate-only
// on imperceptibly small scaleX & scaleY values.
static bool isBasicallyTranslate(const SkMatrix& matrix) {
if (!matrix.isScaleTranslate()) return false;
return MathUtils::isOne(matrix.getScaleX()) && MathUtils::isOne(matrix.getScaleY());
}
static bool shouldFilter(const SkMatrix& matrix) {
if (!matrix.isScaleTranslate()) return true;
// We only care about meaningful scale here
bool noScale = MathUtils::isOne(matrix.getScaleX())
&& MathUtils::isOne(matrix.getScaleY());
bool pixelAligned = SkScalarIsInt(matrix.getTranslateX())
&& SkScalarIsInt(matrix.getTranslateY());
return !(noScale && pixelAligned);
}
bool LayerDrawable::DrawLayer(GrContext* context, SkCanvas* canvas, Layer* layer,
const SkRect* srcRect, const SkRect* dstRect,
bool useLayerTransform) {
@@ -101,7 +120,7 @@ bool LayerDrawable::DrawLayer(GrContext* context, SkCanvas* canvas, Layer* layer
// Integer translation is defined as when src rect and dst rect align fractionally.
// Skia TextureOp has the above logic build-in, but not NonAAFillRectOp. TextureOp works
// only for SrcOver blending and without color filter (readback uses Src blending).
bool isIntegerTranslate = totalMatrix.isTranslate()
bool isIntegerTranslate = isBasicallyTranslate(totalMatrix)
&& SkScalarFraction(skiaDestRect.fLeft + totalMatrix[SkMatrix::kMTransX])
== SkScalarFraction(skiaSrcRect.fLeft)
&& SkScalarFraction(skiaDestRect.fTop + totalMatrix[SkMatrix::kMTransY])
@@ -112,10 +131,7 @@ bool LayerDrawable::DrawLayer(GrContext* context, SkCanvas* canvas, Layer* layer
canvas->drawImageRect(layerImage.get(), skiaSrcRect, skiaDestRect, &paint,
SkCanvas::kFast_SrcRectConstraint);
} else {
bool isIntegerTranslate = totalMatrix.isTranslate()
&& SkScalarIsInt(totalMatrix[SkMatrix::kMTransX])
&& SkScalarIsInt(totalMatrix[SkMatrix::kMTransY]);
if (layer->getForceFilter() || !isIntegerTranslate) {
if (layer->getForceFilter() || shouldFilter(totalMatrix)) {
paint.setFilterQuality(kLow_SkFilterQuality);
}
canvas->drawImage(layerImage.get(), 0, 0, &paint);

4
libs/hwui/utils/MathUtils.h
View File

@@ -34,6 +34,10 @@ public:
return (value >= -NON_ZERO_EPSILON) && (value <= NON_ZERO_EPSILON);
}
inline static bool isOne(float value) {
return areEqual(value, 1.0f);
}
inline static bool isPositive(float value) { return value >= NON_ZERO_EPSILON; }
/**