Add autoscaling test and setHeight method

setHeight method taking UnitValue
Tests for autoscaling

DEVSIX-1658

Author: Samuel Huylebroeck

layout/src/main/java/com/itextpdf/layout/ElementPropertyContainer.java

@@ -188,6 +188,16 @@ public T setHeight(float height) {
         return (T) (Object) this;
     }
 
+    /**
+     * Sets the width property of the Element with a {@link UnitValue}.
+     *
+     * @param height a {@link UnitValue} object
+     * @return this Element.
+     */
+    public T setHeight(UnitValue height) {
+        setProperty(Property.HEIGHT, height);
+        return (T) (Object) this;
+    }
     /**
      * Sets values for a relative repositioning of the Element. Also has as a
      * side effect that the Element's {@link Property#POSITION} is changed to

layout/src/test/java/com/itextpdf/layout/ImageTest.java

@@ -471,6 +471,78 @@ public void imageTest17() throws IOException, InterruptedException {
         Assert.assertNull(new CompareTool().compareByContent(outFileName, cmpFileName, destinationFolder, "diff"));
     }
 
+    @Test
+    public void imageTest18() throws IOException, InterruptedException {
+        String outFileName = destinationFolder + "imageTest18.pdf";
+        String cmpFileName = sourceFolder + "cmp_imageTest18.pdf";
+
+        PdfWriter writer = new PdfWriter(outFileName);
+        PdfDocument pdfDoc = new PdfDocument(writer);
+        Document doc = new Document(pdfDoc);
+
+        Image image = new Image(ImageDataFactory.create(sourceFolder + "Desert.jpg"));
+        image.setAutoScale(true);
+
+        Div container = new Div();
+        container.setBorder(new SolidBorder(1f));
+        container.setWidth(UnitValue.createPercentValue(50f));
+        container.setHeight(UnitValue.createPointValue(300f));
+        container.add(image);
+        doc.add(container);
+
+        doc.close();
+
+        Assert.assertNull(new CompareTool().compareByContent(outFileName, cmpFileName, destinationFolder, "diff"));
+    }
+    @Ignore("DEVSIX-1658")
+    @Test
+    public void imageTest19() throws IOException, InterruptedException {
+        String outFileName = destinationFolder + "imageTest19.pdf";
+        String cmpFileName = sourceFolder + "cmp_imageTest19.pdf";
+
+        PdfWriter writer = new PdfWriter(outFileName);
+        PdfDocument pdfDoc = new PdfDocument(writer);
+        Document doc = new Document(pdfDoc);
+
+        Image image = new Image(ImageDataFactory.create(sourceFolder + "Desert.jpg"));
+        image.setAutoScaleHeight(true);
+
+        Div container = new Div();
+        container.setBorder(new SolidBorder(1f));
+        container.setWidth(UnitValue.createPercentValue(50f));
+        container.setHeight(UnitValue.createPointValue(300f));
+        container.add(image);
+        doc.add(container);
+
+        doc.close();
+
+        Assert.assertNull(new CompareTool().compareByContent(outFileName, cmpFileName, destinationFolder, "diff"));
+    }
+
+    @Test
+    public void imageTest20() throws IOException, InterruptedException {
+        String outFileName = destinationFolder + "imageTest20.pdf";
+        String cmpFileName = sourceFolder + "cmp_imageTest20.pdf";
+
+        PdfWriter writer = new PdfWriter(outFileName);
+        PdfDocument pdfDoc = new PdfDocument(writer);
+        Document doc = new Document(pdfDoc);
+
+        Image image = new Image(ImageDataFactory.create(sourceFolder + "Desert.jpg"));
+        image.setAutoScaleWidth(true);
+
+        Div container = new Div();
+        container.setBorder(new SolidBorder(1f));
+        container.setWidth(UnitValue.createPercentValue(60f));
+        container.setHeight(UnitValue.createPointValue(300f));
+        container.add(image);
+        doc.add(container);
+
+        doc.close();
+
+        Assert.assertNull(new CompareTool().compareByContent(outFileName, cmpFileName, destinationFolder, "diff"));
+    }
+
     /**
      * Image can be reused in layout, so flushing it on the very first draw is a bad thing.
      */