Consider not set LINE_HEIGHT property to be exactly the same as LineHeight.NORMAL

DEVSIX-3662

Author: yulian-gaponenko

layout/src/main/java/com/itextpdf/layout/renderer/LineRenderer.java

@@ -129,7 +129,6 @@ public LayoutResult layout(LayoutContext layoutContext) {
 
         float curWidth = 0;
         if (RenderingMode.HTML_MODE.equals(this.<RenderingMode>getProperty(Property.RENDERING_MODE))
-                && hasProperty(Property.LINE_HEIGHT)
                 && hasChildRendererInHtmlMode()) {
             float[] ascenderDescender = LineHeightHelper.getActualAscenderDescender(this);
             maxAscent = ascenderDescender[0];

layout/src/test/java/com/itextpdf/layout/renderer/LineRendererUnitTest.java

@@ -43,6 +43,8 @@ This file is part of the iText (R) project.
 package com.itextpdf.layout.renderer;
 
 import com.itextpdf.io.LogMessageConstant;
+import com.itextpdf.io.font.constants.StandardFonts;
+import com.itextpdf.kernel.font.PdfFontFactory;
 import com.itextpdf.kernel.geom.Rectangle;
 import com.itextpdf.kernel.pdf.xobject.PdfFormXObject;
 import com.itextpdf.layout.Document;
@@ -61,6 +63,7 @@ This file is part of the iText (R) project.
 import com.itextpdf.test.annotations.LogMessages;
 import com.itextpdf.test.annotations.type.UnitTest;
 
+import java.io.IOException;
 import java.util.Arrays;
 import org.junit.Assert;
 import org.junit.Test;
@@ -279,4 +282,27 @@ public void lineRendererLayoutInHtmlModeWithLineHeightAndChildInHtmlModeTest() {
         Assert.assertEquals(28.67920f, lineRenderer.maxAscent, EPS);
         Assert.assertEquals(-21.32080f, lineRenderer.maxDescent, EPS);
     }
+
+    @Test
+    public void lineRendererLayoutInHtmlModeWithLineHeightPropertyNotSet() throws IOException {
+        LineRenderer lineRenderer = new LineRenderer();
+        lineRenderer.setParent(createDocument().getRenderer());
+        lineRenderer.setProperty(Property.RENDERING_MODE, RenderingMode.HTML_MODE);
+
+        // Set fonts with different ascent/descent to line and text
+        lineRenderer.setProperty(Property.FONT, PdfFontFactory.createFont(StandardFonts.HELVETICA));
+
+        TextRenderer textRenderer = new TextRenderer(new Text("text"));
+        textRenderer.setProperty(Property.FONT, PdfFontFactory.createFont(StandardFonts.COURIER));
+
+        lineRenderer.addChild(textRenderer);
+        LayoutResult layoutResLineHeightNotSet = lineRenderer.layout(new LayoutContext(createLayoutArea(1000, 1000)));
+
+        lineRenderer.setProperty(Property.LINE_HEIGHT, LineHeight.createNormalValue());
+        LayoutResult layoutResLineHeightNormal = lineRenderer.layout(new LayoutContext(createLayoutArea(1000, 1000)));
+
+        Rectangle bboxLineHeightNotSet = layoutResLineHeightNotSet.getOccupiedArea().getBBox();
+        Rectangle bboxLineHeightNormal = layoutResLineHeightNormal.getOccupiedArea().getBBox();
+        Assert.assertTrue(bboxLineHeightNotSet.equalsWithEpsilon(bboxLineHeightNormal));
+    }
 }