Added resizing with keeping aspect ratio when dimension is 0.

Author: mcharmas

picasso/src/main/java/com/squareup/picasso/BitmapHunter.java

@@ -388,12 +388,15 @@ static Bitmap transformResult(Request data, Bitmap result, int exifRotation) {
         float heightRatio = targetHeight / (float) inHeight;
         float scale = widthRatio < heightRatio ? widthRatio : heightRatio;
         matrix.preScale(scale, scale);
-      } else if (targetWidth != 0 && targetHeight != 0 //
+      } else if ((targetWidth != 0 || targetHeight != 0) //
           && (targetWidth != inWidth || targetHeight != inHeight)) {
         // If an explicit target size has been specified and they do not match the results bounds,
         // pre-scale the existing matrix appropriately.
-        float sx = targetWidth / (float) inWidth;
-        float sy = targetHeight / (float) inHeight;
+        // Keep aspect ratio if one dimension is set to 0.
+        float sx = targetWidth != 0 ? targetWidth / (float) inWidth
+            : targetHeight / (float) inHeight;
+        float sy = targetHeight != 0 ? targetHeight / (float) inHeight
+            : targetWidth / (float) inWidth;
         matrix.preScale(sx, sy);
       }
     }

picasso/src/main/java/com/squareup/picasso/Request.java

@@ -152,15 +152,15 @@ String getName() {
   }
 
   public boolean hasSize() {
-    return targetWidth != 0;
+    return targetWidth != 0 || targetHeight != 0;
   }
 
   boolean needsTransformation() {
     return needsMatrixTransform() || hasCustomTransformations();
   }
 
   boolean needsMatrixTransform() {
-    return targetWidth != 0 || rotationDegrees != 0;
+    return hasSize() || rotationDegrees != 0;
   }
 
   boolean hasCustomTransformations() {
@@ -223,7 +223,7 @@ boolean hasImage() {
     }
 
     boolean hasSize() {
-      return targetWidth != 0;
+      return targetWidth != 0 || targetHeight != 0;
     }
 
     /**
@@ -254,13 +254,19 @@ public Builder setResourceId(int resourceId) {
       return this;
     }
 
-    /** Resize the image to the specified size in pixels. */
+    /**
+     * Resize the image to the specified size in pixels.
+     * Use 0 as desired dimension to resize keeping aspect ratio.
+     */
     public Builder resize(int targetWidth, int targetHeight) {
-      if (targetWidth <= 0) {
-        throw new IllegalArgumentException("Width must be positive number.");
+      if (targetWidth < 0) {
+        throw new IllegalArgumentException("Width must be positive number or 0.");
+      }
+      if (targetHeight < 0) {
+        throw new IllegalArgumentException("Height must be positive number or 0.");
       }
-      if (targetHeight <= 0) {
-        throw new IllegalArgumentException("Height must be positive number.");
+      if (targetHeight == 0 && targetWidth == 0) {
+        throw new IllegalArgumentException("At least one dimension has to be positive number.");
       }
       this.targetWidth = targetWidth;
       this.targetHeight = targetHeight;
@@ -364,11 +370,13 @@ public Request build() {
       if (centerInside && centerCrop) {
         throw new IllegalStateException("Center crop and center inside can not be used together.");
       }
-      if (centerCrop && targetWidth == 0) {
-        throw new IllegalStateException("Center crop requires calling resize.");
+      if (centerCrop && (targetWidth == 0 || targetHeight == 0)) {
+        throw new IllegalStateException(
+            "Center crop requires calling resize with positive width and height.");
       }
-      if (centerInside && targetWidth == 0) {
-        throw new IllegalStateException("Center inside requires calling resize.");
+      if (centerInside && (targetWidth == 0 || targetHeight == 0)) {
+        throw new IllegalStateException(
+            "Center inside requires calling resize with positive width and height.");
       }
       return new Request(uri, resourceId, transformations, targetWidth, targetHeight, centerCrop,
           centerInside, rotationDegrees, rotationPivotX, rotationPivotY, hasRotationPivot, config);

picasso/src/main/java/com/squareup/picasso/RequestHandler.java

@@ -143,8 +143,20 @@ static void calculateInSampleSize(int reqWidth, int reqHeight, int width, int he
                                     BitmapFactory.Options options) {
     int sampleSize = 1;
     if (height > reqHeight || width > reqWidth) {
-      final int heightRatio = (int) Math.floor((float) height / (float) reqHeight);
-      final int widthRatio = (int) Math.floor((float) width / (float) reqWidth);
+      final int heightRatio;
+      final int widthRatio;
+      if (reqHeight == 0) {
+        int ratio = (int) Math.floor((float) width / (float) reqWidth);
+        heightRatio = ratio;
+        widthRatio = ratio;
+      } else if (reqWidth == 0) {
+        int ratio = (int) Math.floor((float) height / (float) reqHeight);
+        heightRatio = ratio;
+        widthRatio = ratio;
+      } else {
+        heightRatio = (int) Math.floor((float) height / (float) reqHeight);
+        widthRatio = (int) Math.floor((float) width / (float) reqWidth);
+      }
       sampleSize = heightRatio < widthRatio ? heightRatio : widthRatio;
     }
     options.inSampleSize = sampleSize;

picasso/src/main/java/com/squareup/picasso/Utils.java

@@ -179,7 +179,7 @@ static String createKey(Request data, StringBuilder builder) {
       }
       builder.append('\n');
     }
-    if (data.targetWidth != 0) {
+    if (data.hasSize()) {
       builder.append("resize:").append(data.targetWidth).append('x').append(data.targetHeight);
       builder.append('\n');
     }

picasso/src/test/java/com/squareup/picasso/RequestCreatorTest.java

@@ -432,6 +432,30 @@ public void intoRemoteViewsNotificationWithFitThrows() throws Exception {
     }
   }
 
+  @Test
+  public void intoTargetResizeWith0WithCenterInsideOrCenterCropThrows() throws Exception {
+    try {
+      new RequestCreator(picasso, URI_1, 0).resize(0, 10).centerInside().into(mockTarget());
+      fail("Center inside with unknown width should throw exception.");
+    } catch (IllegalStateException expected) {
+    }
+    try {
+      new RequestCreator(picasso, URI_1, 0).resize(10, 0).centerInside().into(mockTarget());
+      fail("Center inside with unknown height should throw exception.");
+    } catch (IllegalStateException expected) {
+    }
+    try {
+      new RequestCreator(picasso, URI_1, 0).resize(0, 10).centerCrop().into(mockTarget());
+      fail("Center inside with unknown width should throw exception.");
+    } catch (IllegalStateException expected) {
+    }
+    try {
+      new RequestCreator(picasso, URI_1, 0).resize(10, 0).centerCrop().into(mockTarget());
+      fail("Center inside with unknown height should throw exception.");
+    } catch (IllegalStateException expected) {
+    }
+  }
+
   @Test public void invalidResize() throws Exception {
     try {
       new RequestCreator().resize(-1, 10);
@@ -444,13 +468,8 @@ public void intoRemoteViewsNotificationWithFitThrows() throws Exception {
     } catch (IllegalArgumentException expected) {
     }
     try {
-      new RequestCreator().resize(0, 10);
-      fail("Zero width should throw exception.");
-    } catch (IllegalArgumentException expected) {
-    }
-    try {
-      new RequestCreator().resize(10, 0);
-      fail("Zero height should throw exception.");
+      new RequestCreator().resize(0, 0);
+      fail("Zero dimensions should throw exception.");
     } catch (IllegalArgumentException expected) {
     }
   }