From ff9133f1125865f4e14a92d4b5e661f1ac92524d Mon Sep 17 00:00:00 2001 From: Tony Wickham Date: Tue, 22 Sep 2015 13:21:20 -0700 Subject: [PATCH] Adjust cropped bitmap bounds to correctly fit in original bitmap bounds. Bug: 22631531 Change-Id: Ia0c1bccf4b8af3a8b335b865974c5d08f9836689 --- .../src/com/android/gallery3d/common/BitmapCropTask.java | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/WallpaperPicker/src/com/android/gallery3d/common/BitmapCropTask.java b/WallpaperPicker/src/com/android/gallery3d/common/BitmapCropTask.java index cb7090fad..a79fb65c9 100644 --- a/WallpaperPicker/src/com/android/gallery3d/common/BitmapCropTask.java +++ b/WallpaperPicker/src/com/android/gallery3d/common/BitmapCropTask.java @@ -293,22 +293,16 @@ public class BitmapCropTask extends AsyncTask { roundedTrueCrop.right = roundedTrueCrop.left + fullSize.getWidth(); } if (roundedTrueCrop.right > fullSize.getWidth()) { - // Adjust the left value - int adjustment = roundedTrueCrop.left - - Math.max(0, roundedTrueCrop.right - roundedTrueCrop.width()); - roundedTrueCrop.left -= adjustment; - roundedTrueCrop.right -= adjustment; + // Adjust the left and right values. + roundedTrueCrop.offset(-(roundedTrueCrop.right - fullSize.getWidth()), 0); } if (roundedTrueCrop.height() > fullSize.getHeight()) { // Adjust the height roundedTrueCrop.bottom = roundedTrueCrop.top + fullSize.getHeight(); } if (roundedTrueCrop.bottom > fullSize.getHeight()) { - // Adjust the top value - int adjustment = roundedTrueCrop.top - - Math.max(0, roundedTrueCrop.bottom - roundedTrueCrop.height()); - roundedTrueCrop.top -= adjustment; - roundedTrueCrop.bottom -= adjustment; + // Adjust the top and bottom values. + roundedTrueCrop.offset(0, -(roundedTrueCrop.bottom - fullSize.getHeight())); } crop = Bitmap.createBitmap(fullSize, roundedTrueCrop.left, -- 2.11.0