Rendering text-justify:distribute for 8 bit characters.

Source/platform/fonts/mac/ComplexTextController.cpp

Index: Source/platform/fonts/mac/ComplexTextController.cpp
diff --git a/Source/platform/fonts/mac/ComplexTextController.cpp b/Source/platform/fonts/mac/ComplexTextController.cpp
index 4537d16d5de528f220f87f2441e769fe45e2c418..c8312e7435da2763f6622abe1b43cb4282d090d3 100644
--- a/Source/platform/fonts/mac/ComplexTextController.cpp
+++ b/Source/platform/fonts/mac/ComplexTextController.cpp
@@ -76,16 +76,18 @@ ComplexTextController::ComplexTextController(const Font* font, const TextRun& ru
     , m_minGlyphBoundingBoxY(numeric_limits<float>::max())
     , m_maxGlyphBoundingBoxY(numeric_limits<float>::min())
     , m_lastRoundingGlyph(0)
+    , m_distributeJustification(false)

[leviw_travelin_and_unemployed, 2014/07/28 23:59:58]

Why are you making this a member variable if you only pass it on to
Character::expansionOpportunityCount? You only need a local variable.

 {
     if (!m_expansion)
         m_expansionPerOpportunity = 0;
     else {
         bool isAfterExpansion = m_afterExpansion;
         unsigned expansionOpportunityCount;
+        m_distributeJustification = m_run.isDistributeJustification();
         if (m_run.is8Bit())
-            expansionOpportunityCount = Character::expansionOpportunityCount(m_run.characters8(), m_end, m_run.ltr() ? LTR : RTL, isAfterExpansion);
+            expansionOpportunityCount = Character::expansionOpportunityCount(m_run.characters8(), m_end, m_run.ltr() ? LTR : RTL, isAfterExpansion, m_distributeJustification);
          else
-            expansionOpportunityCount = Character::expansionOpportunityCount(m_run.characters16(), m_end, m_run.ltr() ? LTR : RTL, isAfterExpansion);
+            expansionOpportunityCount = Character::expansionOpportunityCount(m_run.characters16(), m_end, m_run.ltr() ? LTR : RTL, isAfterExpansion, m_distributeJustification);
         if (isAfterExpansion && !m_run.allowsTrailingExpansion())
             expansionOpportunityCount--;