Add small content stream optimization for ActualTextIterator

DEVSIX-1983

Author: Snipx

io/src/main/java/com/itextpdf/io/font/otf/ActualTextIterator.java

@@ -79,14 +79,18 @@ public GlyphLine.GlyphLinePart next() {
                 return null;
             }
             pos = currentResult.end;
-            while (pos < glyphLine.end && !glyphLinePartNeedsActualText(currentResult)) {
+
+            if (!glyphLinePartNeedsActualText(currentResult)) {
                 currentResult.actualText = null;
-                GlyphLine.GlyphLinePart nextResult = nextGlyphLinePart(pos);
-                if (nextResult != null && !glyphLinePartNeedsActualText(nextResult)) {
-                    currentResult.end = nextResult.end;
-                    pos = nextResult.end;
-                } else {
-                    break;
+                // Try to add more pieces without "actual text"
+                while (pos < glyphLine.end) {
+                    GlyphLine.GlyphLinePart nextResult = nextGlyphLinePart(pos);
+                    if (nextResult != null && !glyphLinePartNeedsActualText(nextResult)) {
+                        currentResult.end = nextResult.end;
+                        pos = nextResult.end;
+                    } else {
+                        break;
+                    }
                 }
             }
             return currentResult;

io/src/main/java/com/itextpdf/io/font/otf/GlyphLine.java

@@ -369,6 +369,7 @@ private void addAllGlyphs(int index, List<Glyph> additionalGlyphs) {
     public static class GlyphLinePart {
         public int start;
         public int end;
+        // Might be null if it's not necessary
         public String actualText;
         public boolean reversed;
 

io/src/test/java/com/itextpdf/io/font/otf/ActualTextIteratorTest.java

@@ -0,0 +1,24 @@
+package com.itextpdf.io.font.otf;
+
+import com.itextpdf.test.annotations.type.UnitTest;
+import org.junit.Assert;
+import org.junit.Test;
+import org.junit.experimental.categories.Category;
+
+import java.util.Arrays;
+
+@Category(UnitTest.class)
+public class ActualTextIteratorTest {
+
+    @Test
+    public void testActualTestParts() {
+        Glyph glyph = new Glyph(200, 200, '\u002d');
+        GlyphLine glyphLine = new GlyphLine(Arrays.asList(glyph));
+        glyphLine.setActualText(0, 1, "\u002d");
+        ActualTextIterator actualTextIterator = new ActualTextIterator(glyphLine);
+        GlyphLine.GlyphLinePart part = actualTextIterator.next();
+        // When actual text is the same as the result by text extraction, we should omit redundant actual text in the content stream
+        Assert.assertNull(part.actualText);
+    }
+
+}