Support overflow-wrap: break-word/anywhere in case of the not fitting single glyph

DEVSIX-1438

Author: yulian-gaponenko

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

@@ -1871,10 +1871,12 @@ LastFittingChildRendererData getIndexAndLayoutResultOfTheLastTextRendererWithNoS
         }
 
         if (lastAnalyzedTextLayoutResult == null) {
-            OverflowWrapPropertyValue overflowWrapPropertyValue =
+            OverflowWrapPropertyValue overflowWrapValue =
                     childRenderers.get(childPos).<OverflowWrapPropertyValue>getProperty(Property.OVERFLOW_WRAP);
-            if (overflowWrapPropertyValue == OverflowWrapPropertyValue.ANYWHERE
-                    || overflowWrapPropertyValue == OverflowWrapPropertyValue.BREAK_WORD
+            boolean overflowWrapNotNormal = overflowWrapValue == OverflowWrapPropertyValue.ANYWHERE
+                    || overflowWrapValue == OverflowWrapPropertyValue.BREAK_WORD;
+            if (overflowWrapNotNormal
+                        && textSequenceLayoutResults.get(lastAnalyzedTextRenderer).getStatus() != LayoutResult.NOTHING
                     || isOverflowFit) {
                 lastAnalyzedTextRenderer = childPos;
                 lastAnalyzedTextLayoutResult = textSequenceLayoutResults.get(lastAnalyzedTextRenderer);
@@ -1949,10 +1951,11 @@ LastFittingChildRendererData getIndexAndLayoutResultOfTheLastTextRendererWithNoS
             // otherwise return null as a flag to move forward across this.childRenderers
             // till the end of the unbreakable word
             if (status == SpecialScriptsContainingSequenceStatus.FORCED_SPLIT) {
-                OverflowWrapPropertyValue overflowWrapPropertyValue =
+                OverflowWrapPropertyValue overflowWrapValue =
                         childRenderers.get(childPos).<OverflowWrapPropertyValue>getProperty(Property.OVERFLOW_WRAP);
-                if (overflowWrapPropertyValue == OverflowWrapPropertyValue.ANYWHERE
-                        || overflowWrapPropertyValue == OverflowWrapPropertyValue.BREAK_WORD
+                boolean overflowWrapNotNormal = overflowWrapValue == OverflowWrapPropertyValue.ANYWHERE
+                        || overflowWrapValue == OverflowWrapPropertyValue.BREAK_WORD;
+                if (overflowWrapNotNormal && childPosLayoutResult.getStatus() != LayoutResult.NOTHING
                         || isOverflowFit) {
                     if (childPosLayoutResult.getStatus() != LayoutResult.NOTHING) {
                         returnLayoutResult = childPosLayoutResult;

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

@@ -183,8 +183,9 @@ public LayoutResult layout(LayoutContext layoutContext) {
         OverflowPropertyValue overflowX = this.parent.<OverflowPropertyValue>getProperty(Property.OVERFLOW_X);
 
         OverflowWrapPropertyValue overflowWrap = this.<OverflowWrapPropertyValue>getProperty(Property.OVERFLOW_WRAP);
-        if (overflowWrap == OverflowWrapPropertyValue.ANYWHERE
-                || overflowWrap == OverflowWrapPropertyValue.BREAK_WORD) {
+        boolean overflowWrapNotNormal = overflowWrap == OverflowWrapPropertyValue.ANYWHERE
+                || overflowWrap == OverflowWrapPropertyValue.BREAK_WORD;
+        if (overflowWrapNotNormal) {
             overflowX = OverflowPropertyValue.FIT;
         }
 
@@ -378,9 +379,13 @@ public LayoutResult layout(LayoutContext layoutContext) {
                         && firstCharacterWhichExceedsAllowedWidth == -1
                         || ind == specialScriptFirstNotFittingIndex) {
                     firstCharacterWhichExceedsAllowedWidth = ind;
-                    if (TextUtil.isSpaceOrWhitespace(text.get(ind))) {
-                        containsPossibleBreak = true;
-                        wordBreakGlyphAtLineEnding = currentGlyph;
+                    boolean spaceOrWhitespace = TextUtil.isSpaceOrWhitespace(text.get(ind));
+                    OverflowPropertyValue parentOverflowX = parent.<OverflowPropertyValue>getProperty(Property.OVERFLOW_X);
+                    if (spaceOrWhitespace || overflowWrapNotNormal && !isOverflowFit(parentOverflowX)) {
+                        if (spaceOrWhitespace) {
+                            containsPossibleBreak = true;
+                            wordBreakGlyphAtLineEnding = currentGlyph;
+                        }
                         if (ind == firstPrintPos) {
                             forcePartialSplitOnFirstChar = true;
                             firstCharacterWhichExceedsAllowedWidth = ind + 1;
@@ -1242,7 +1247,7 @@ public static float[] calculateAscenderDescender(PdfFont font, RenderingMode mod
         }
         return new float[] {ascender, descender};
     }
-    
+
     List<int[]> getReversedRanges() {
         return reversedRanges;
     }

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

@@ -43,6 +43,7 @@ This file is part of the iText (R) project.
 import com.itextpdf.layout.element.Text;
 import com.itextpdf.layout.property.FloatPropertyValue;
 import com.itextpdf.layout.property.OverflowPropertyValue;
+import com.itextpdf.layout.property.OverflowWrapPropertyValue;
 import com.itextpdf.layout.property.Property;
 import com.itextpdf.layout.property.RenderingMode;
 import com.itextpdf.layout.property.TextAlignment;
@@ -657,4 +658,25 @@ public void minMaxWidthWordSplitAcrossMultipleTextRenderers() throws IOException
 
         Assert.assertNull(new CompareTool().compareByContent(outFileName, cmpFileName, destinationFolder));
     }
+
+    @Test
+    public void overflowWrapBreakWordWithOverflowXTest() throws IOException, InterruptedException {
+        String outFileName = destinationFolder + "overflowWrapBreakWordWithOverflowXTest.pdf";
+        String cmpFileName = sourceFolder + "cmp_overflowWrapBreakWordWithOverflowXTest.pdf";
+        PdfDocument pdfDocument = new PdfDocument(new PdfWriter(outFileName));
+        Document doc = new Document(pdfDocument);
+        doc.setFontSize(40);
+        Text text = new Text("wow");
+        Paragraph paragraph = new Paragraph()
+                .add(text)
+                .setBackgroundColor(ColorConstants.YELLOW)
+                .setWidth(10)
+                .setBorder(new SolidBorder(1));
+        paragraph.setProperty(Property.OVERFLOW_X, OverflowPropertyValue.VISIBLE);
+        paragraph.setProperty(Property.OVERFLOW_WRAP, OverflowWrapPropertyValue.BREAK_WORD);
+        paragraph.setProperty(Property.RENDERING_MODE, RenderingMode.HTML_MODE);
+        doc.add(paragraph);
+        doc.close();
+        Assert.assertNull(new CompareTool().compareByContent(outFileName, cmpFileName, destinationFolder));
+    }
 }