Rendering text-justify:distribute for 8 bit characters.

Source/core/platform/graphics/Font.h

Index: Source/core/platform/graphics/Font.h
diff --git a/Source/core/platform/graphics/Font.h b/Source/core/platform/graphics/Font.h
index f353f814cfadf49cbff4e09aa55f846950ef25e5..44553e9f5ddee64e921463d2535bbebee7215f74 100644
--- a/Source/core/platform/graphics/Font.h
+++ b/Source/core/platform/graphics/Font.h
@@ -156,8 +156,8 @@ public:
     static bool isCJKIdeograph(UChar32);
     static bool isCJKIdeographOrSymbol(UChar32);
 
-    static unsigned expansionOpportunityCount(const LChar*, size_t length, TextDirection, bool& isAfterExpansion);
-    static unsigned expansionOpportunityCount(const UChar*, size_t length, TextDirection, bool& isAfterExpansion);
+    static unsigned expansionOpportunityCount(const LChar*, size_t length, TextDirection, bool& isAfterExpansion, bool distributeJustification = false);

[leviw_travelin_and_unemployed, 2013/11/01 20:39:14]

Yech, boolean params! Why not just use the TextJustify enum?

[dw.im, 2013/11/02 03:13:01]

Because.... I only need to check text-justify is distribute or not in the
function.
Textjustify enum could be one choice here.

Will try and see.

[dw.im, 2013/11/04 02:11:09]

I used boolean parameter here again, 
because including "core/rendering/style/RenderStyleConstants.h" to use the
TextJustify enum is illegal for this file.
presubmit detect this as Illegal include.

Anyway, I only need to check the value is distribute or not in this function, 
so, how about leaving the boolean parameter with its name so that readers can
know the meaning of the parameter?

+    static unsigned expansionOpportunityCount(const UChar*, size_t length, TextDirection, bool& isAfterExpansion, bool distributeJustification = false);
 
     static void setShouldUseSmoothing(bool);
     static bool shouldUseSmoothing();