Fix integer truncation in Character test functions

third_party/WebKit/Source/platform/fonts/Character.h

 /*
  * Copyright (C) 2014 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 57 matching lines @@
     static unsigned expansionOpportunityCount(const UChar*, size_t length, TextDirection, bool& isAfterExpansion, const TextJustify);
     static unsigned expansionOpportunityCount(const TextRun& run, bool& isAfterExpansion)
     {
         if (run.is8Bit())
             return expansionOpportunityCount(run.characters8(), run.length(), run.direction(), isAfterExpansion, run.getTextJustify());
         return expansionOpportunityCount(run.characters16(), run.length(), run.direction(), isAfterExpansion, run.getTextJustify());
     }
 
     static bool isUprightInMixedVertical(UChar32 character);
 
-    static bool treatAsSpace(UChar c)
+    static bool treatAsSpace(UChar32 c)
     {
         return c == spaceCharacter
             || c == tabulationCharacter
             || c == newlineCharacter
             || c == noBreakSpaceCharacter;
     }
-    static bool treatAsZeroWidthSpace(UChar c)
+    static bool treatAsZeroWidthSpace(UChar32 c)
     {
         return treatAsZeroWidthSpaceInComplexScript(c)
             || c == zeroWidthNonJoinerCharacter
             || c == zeroWidthJoinerCharacter;
     }
-    static bool treatAsZeroWidthSpaceInComplexScript(UChar c)
+    static bool treatAsZeroWidthSpaceInComplexScript(UChar32 c)
     {
         return c < 0x20 // ASCII Control Characters
             || (c >= 0x7F && c < 0xA0) // ASCII Delete .. No-break spaceCharacter
             || c == softHyphenCharacter
             || c == zeroWidthSpaceCharacter
             || (c >= leftToRightMarkCharacter && c <= rightToLeftMarkCharacter)
             || (c >= leftToRightEmbedCharacter && c <= rightToLeftOverrideCharacter)
             || c == zeroWidthNoBreakSpaceCharacter
             || c == objectReplacementCharacter;
     }
@@ skipping 10 matching lines @@
     static bool isEmojiModifierBase(UChar32);
     static bool isEmojiKeycapBase(UChar32);
     static bool isRegionalIndicator(UChar32);
     static bool isModifier(UChar32 c)
     {
         return c >= 0x1F3FB && c <= 0x1F3FF;
     }
 
     static inline UChar normalizeSpaces(UChar character)
     {
+
         if (treatAsSpace(character))
             return spaceCharacter;
 
         if (treatAsZeroWidthSpace(character))
             return zeroWidthSpaceCharacter;
 
         return character;
     }
 
-    static inline bool isNormalizedCanvasSpaceCharacter(UChar c)
+    static inline bool isNormalizedCanvasSpaceCharacter(UChar32 c)
     {
         // According to specification all space characters should be replaced with 0x0020 space character.
         // http://www.whatwg.org/specs/web-apps/current-work/multipage/the-canvas-element.html#text-preparation-algorithm
         // The space characters according to specification are : U+0020, U+0009, U+000A, U+000C, and U+000D.
         // http://www.whatwg.org/specs/web-apps/current-work/multipage/common-microsyntaxes.html#space-character
         // This function returns true for 0x000B also, so that this is backward compatible.
         // Otherwise, the test LayoutTests/canvas/philip/tests/2d.text.draw.space.collapse.space.html will fail
         return c == 0x0009 || (c >= 0x000A && c <= 0x000D);
     }
 
     static String normalizeSpaces(const LChar*, unsigned length);
     static String normalizeSpaces(const UChar*, unsigned length);
 
     static bool isCommonOrInheritedScript(UChar32);
 };
 
 } // namespace blink
 
 #endif