Commit d11be2ad authored by ckitagawa's avatar ckitagawa Committed by Commit Bot

[Paint Preview] Fix positioning on Android M and earlier

Found out that setMatrix() is broken, but concat() is a viable
alternative.

https://stackoverflow.com/questions/17100355/canvas-is-offset-by-setting-identity-matrix-in-ondraw-of-custom-view

Bug: 1126541
Change-Id: I30f4e37dd488385c2f024ac9ae8aa0de8277aa6d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461214Reviewed-by: default avatarMehran Mahmoudi <mahmoudi@chromium.org>
Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org>
Cr-Commit-Position: refs/heads/master@{#815354}
parent 150c2a56
......@@ -98,7 +98,7 @@ class PlayerFrameView extends FrameLayout {
@Override
protected void onDraw(Canvas canvas) {
canvas.save();
if (!mScaleMatrix.isIdentity()) canvas.setMatrix(mScaleMatrix);
canvas.concat(mScaleMatrix);
mBitmapPainter.onDraw(canvas);
canvas.restore();
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment