danfickle
diff --git a/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/css/style/CssContext.java
Copy file name to clipboard
+6Lines changed: 6 additions & 0 deletions b/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/css/style/CssContext.java
Copy file name to clipboard
+6Lines changed: 6 additions & 0 deletions
diff --git a/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/extend/TextRenderer.java
Copy file name to clipboard
+5-9Lines changed: 5 additions & 9 deletions b/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/extend/TextRenderer.java
Copy file name to clipboard
+5-9Lines changed: 5 additions & 9 deletions
diff --git a/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/layout/BoxBuilder.java
Copy file name to clipboard
+2-2Lines changed: 2 additions & 2 deletions b/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/layout/BoxBuilder.java
Copy file name to clipboard
+2-2Lines changed: 2 additions & 2 deletions
diff --git a/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/layout/Breaker.java
Copy file name to clipboard
+16-7Lines changed: 16 additions & 7 deletions b/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/layout/Breaker.java
Copy file name to clipboard
+16-7Lines changed: 16 additions & 7 deletions
diff --git a/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/layout/InlineBoxing.java
Copy file name to clipboard
-2Lines changed: 0 additions & 2 deletions b/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/layout/InlineBoxing.java
Copy file name to clipboard
-2Lines changed: 0 additions & 2 deletions
diff --git a/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/layout/LayoutContext.java
Copy file name to clipboard
+2Lines changed: 2 additions & 0 deletions b/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/layout/LayoutContext.java
Copy file name to clipboard
+2Lines changed: 2 additions & 0 deletions
diff --git a/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/layout/LineBreakContext.java
Copy file name to clipboard
-9Lines changed: 0 additions & 9 deletions b/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/layout/LineBreakContext.java
Copy file name to clipboard
-9Lines changed: 0 additions & 9 deletions
diff --git a/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/render/InlineBox.java
Copy file name to clipboard
+1-10Lines changed: 1 addition & 10 deletions b/‎openhtmltopdf-core/src/main/java/com/openhtmltopdf/render/InlineBox.java
Copy file name to clipboard
+1-10Lines changed: 1 addition & 10 deletions
@@ -2,6 +2,8 @@
 
 import com.openhtmltopdf.context.StyleReference;
 import com.openhtmltopdf.css.value.FontSpecification;
+import com.openhtmltopdf.extend.FontContext;
+import com.openhtmltopdf.extend.TextRenderer;
 import com.openhtmltopdf.render.FSFont;
 import com.openhtmltopdf.render.FSFontMetrics;
 
@@ -29,4 +31,8 @@ public interface CssContext {
     StyleReference getCss();
 
     FSFontMetrics getFSFontMetrics(FSFont font);
+    
+    FontContext getFontContext();
+    
+    TextRenderer getTextRenderer();
 }
@@ -20,8 +20,7 @@
  */
 package com.openhtmltopdf.extend;
 
-import java.awt.Rectangle;
-
+import com.openhtmltopdf.layout.Breaker;
 import com.openhtmltopdf.render.FSFont;
 import com.openhtmltopdf.render.FSFontMetrics;
 import com.openhtmltopdf.render.JustificationInfo;
@@ -32,17 +31,14 @@ public interface TextRenderer {
     public void drawString(OutputDevice outputDevice, String string, float x, float y);
     public void drawString(
             OutputDevice outputDevice, String string, float x, float y, JustificationInfo info);
-    
-    public void drawGlyphVector(OutputDevice outputDevice, FSGlyphVector vector, float x, float y);
-    
-    public FSGlyphVector getGlyphVector(OutputDevice outputDevice, FSFont font, String string);
-    
-    public float[] getGlyphPositions(OutputDevice outputDevice, FSFont font, FSGlyphVector fsGlyphVector);
-    public Rectangle getGlyphBounds(OutputDevice outputDevice, FSFont font, FSGlyphVector fsGlyphVector, int index, float x, float y);
 
     public FSFontMetrics getFSFontMetrics(
             FontContext context, FSFont font, String string );
 
+    /**
+     * Rarely need to use this method directly.
+     * Instead favor {@link Breaker} static method instead.
+     */
     public int getWidth(FontContext context, FSFont font, String string);
 
     public void setFontScale(float scale);
 
@@ -858,7 +858,7 @@ private static List<Styleable> createGeneratedContentList(
             }
 
             if (content != null) {
-                InlineBox iB = new InlineBox(content, null);
+                InlineBox iB = new InlineBox(content);
                 iB.setContentFunction(contentFunction);
                 iB.setFunction(function);
                 iB.setElement(element);
@@ -1054,7 +1054,7 @@ private static void addColumnOrColumnGroup(
 
     private static InlineBox createInlineBox(
             String text, Element parent, CalculatedStyle parentStyle, Text node) {
-        InlineBox result = new InlineBox(text, node);
+        InlineBox result = new InlineBox(text);
 
         if (parentStyle.isInline() && ! (parent.getParentNode() instanceof Document)) {
             result.setStyle(parentStyle);
 
@@ -23,6 +23,7 @@
 import com.openhtmltopdf.css.constants.CSSName;
 import com.openhtmltopdf.css.constants.IdentValue;
 import com.openhtmltopdf.css.style.CalculatedStyle;
+import com.openhtmltopdf.css.style.CssContext;
 import com.openhtmltopdf.extend.FSTextBreaker;
 import com.openhtmltopdf.render.FSFont;
 
@@ -72,12 +73,13 @@ public static void breakText(LayoutContext c,
             LineBreakContext context, int avail, CalculatedStyle style) {
         FSFont font = style.getFSFont(c);
         IdentValue whitespace = style.getWhitespace();
+        float letterSpacing = style.hasLetterSpacing() ? 
+                style.getFloatPropertyProportionalWidth(CSSName.LETTER_SPACING, 0, c) : 0f;
 
         // ====== handle nowrap
         if (whitespace == IdentValue.NOWRAP) {
-        	context.setEnd(context.getLast());
-        	context.setWidth(c.getTextRenderer().getWidth(
-                    c.getFontContext(), font, context.getCalculatedSubstring()));
+            context.setEnd(context.getLast());
+            context.setWidth(Breaker.getTextWidthWithLetterSpacing(c, font, context.getCalculatedSubstring(), letterSpacing));
             return;
         }
 
@@ -88,14 +90,12 @@ public static void breakText(LayoutContext c,
             int n = context.getStartSubstring().indexOf(WhitespaceStripper.EOL);
             if (n > -1) {
                 context.setEnd(context.getStart() + n + 1);
-                context.setWidth(c.getTextRenderer().getWidth(
-                        c.getFontContext(), font, context.getCalculatedSubstring()));
+                context.setWidth(Breaker.getTextWidthWithLetterSpacing(c, font, context.getCalculatedSubstring(), letterSpacing));
                 context.setNeedsNewLine(true);
                 context.setEndsOnNL(true);
             } else if (whitespace == IdentValue.PRE) {
             	context.setEnd(context.getLast());
-                context.setWidth(c.getTextRenderer().getWidth(
-                        c.getFontContext(), font, context.getCalculatedSubstring()));
+                context.setWidth(Breaker.getTextWidthWithLetterSpacing(c, font, context.getCalculatedSubstring(), letterSpacing));
             }
         }
 
@@ -281,4 +281,13 @@ public static FSTextBreaker getLineBreakStream(String s, SharedContext shared) {
 		i.setText(s);
 		return i;
 	}
+
+	/**
+	 * Gets the width of a string with letter spacing factored in.
+	 * Favor this method over using the text renderer directly.
+	 */
+    public static int getTextWidthWithLetterSpacing(CssContext c, FSFont font, String text, float letterSpacing) {
+        float extraSpace = text.length() * letterSpacing;
+        return (int) (c.getTextRenderer().getWidth(c.getFontContext(), font, text) + extraSpace);
+    }
 }
@@ -161,7 +161,6 @@ public static void layoutContent(LayoutContext c, BlockBox box, int initialY, in
 
                 LineBreakContext lbContext = new LineBreakContext();
                 lbContext.setMaster(iB.getText());
-                lbContext.setTextNode(iB.getTextNode());
                 if (iB.isDynamicFunction()) {
                     lbContext.setMaster(iB.getContentFunction().getLayoutReplacementText());
                 }
@@ -958,7 +957,6 @@ private static InlineText layoutText(LayoutContext c, CalculatedStyle style, int
         }
 
         result.setMasterText(masterText);
-        result.setTextNode(lbContext.getTextNode());
         result.setSubstring(lbContext.getStart(), lbContext.getEnd());
         result.setWidth(lbContext.getWidth());
         result.setTextDirection(textDirection);
 
@@ -93,6 +93,7 @@ public class LayoutContext implements CssContext {
 
     private Boolean isPrintOverride = null; // True, false, or null for no override.
 
+    @Override
     public TextRenderer getTextRenderer() {
         return _sharedContext.getTextRenderer();
     }
@@ -375,6 +376,7 @@ public ReplacedElementFactory getReplacedElementFactory() {
         return _sharedContext.getReplacedElementFactory();
     }
 
+    @Override
     public FontContext getFontContext() {
         return _fontContext;
     }
 
@@ -36,7 +36,6 @@ public class LineBreakContext {
     private int _width;
     private boolean _endsOnNL;
     private boolean _endsOnSoftHyphen;
-    private Text _textNode;
 
     public int getLast() {
         return _master.length();
@@ -132,14 +131,6 @@ public void setEndsOnNL(boolean b) {
         _endsOnNL = b;
     }
 
-    public Text getTextNode() {
-        return this._textNode;
-    }
-
-    public void setTextNode(Text _text) {
-        this._textNode = _text;
-    }
-
     public boolean isEndsOnSoftHyphen() {
         return this._endsOnSoftHyphen;
     }
 
@@ -21,8 +21,6 @@
 import java.text.BreakIterator;
 
 import org.w3c.dom.Element;
-import org.w3c.dom.Text;
-
 import com.openhtmltopdf.bidi.BidiSplitter;
 import com.openhtmltopdf.css.constants.CSSName;
 import com.openhtmltopdf.css.constants.IdentValue;
@@ -73,12 +71,9 @@ public class InlineBox implements Styleable {
 
     private String _pseudoElementOrClass;
 
-    private final Text _textNode;
-
-    public InlineBox(String text, Text textNode) {
+    public InlineBox(String text) {
         _text = text;
         _originalText = text;
-        _textNode = textNode;
     }
 
     private byte _textDirection;
@@ -503,8 +498,4 @@ public void truncateText() {
         _text = "";
         _originalText = "";
     }
-
-    public Text getTextNode() {
-        return this._textNode;
-    }
 }
Original file line number	Diff line number	Diff line change
`@@ -161,7 +161,6 @@ public static void layoutContent(LayoutContext c, BlockBox box, int initialY, in`
`161`	`161`
`162`	`162`	`LineBreakContext lbContext = new LineBreakContext();`
`163`	`163`	`lbContext.setMaster(iB.getText());`
`164`		`- lbContext.setTextNode(iB.getTextNode());`
`165`	`164`	`if (iB.isDynamicFunction()) {`
`166`	`165`	`lbContext.setMaster(iB.getContentFunction().getLayoutReplacementText());`
`167`	`166`	`}`
`@@ -958,7 +957,6 @@ private static InlineText layoutText(LayoutContext c, CalculatedStyle style, int`
`958`	`957`	`}`
`959`	`958`
`960`	`959`	`result.setMasterText(masterText);`
`961`		`- result.setTextNode(lbContext.getTextNode());`
`962`	`960`	`result.setSubstring(lbContext.getStart(), lbContext.getEnd());`
`963`	`961`	`result.setWidth(lbContext.getWidth());`
`964`	`962`	`result.setTextDirection(textDirection);`
Original file line number	Diff line number	Diff line change
`@@ -93,6 +93,7 @@ public class LayoutContext implements CssContext {`
`93`	`93`
`94`	`94`	`private Boolean isPrintOverride = null; // True, false, or null for no override.`
`95`	`95`
	`96`	`+ @Override`
`96`	`97`	`public TextRenderer getTextRenderer() {`
`97`	`98`	`return _sharedContext.getTextRenderer();`
`98`	`99`	`}`
`@@ -375,6 +376,7 @@ public ReplacedElementFactory getReplacedElementFactory() {`
`375`	`376`	`return _sharedContext.getReplacedElementFactory();`
`376`	`377`	`}`
`377`	`378`
	`379`	`+ @Override`
`378`	`380`	`public FontContext getFontContext() {`
`379`	`381`	`return _fontContext;`
`380`	`382`	`}`