Avoid re-parsing of string in fillText and measureText in Canvas

Source/platform/fonts/WidthIterator.h

 /*
  * Copyright (C) 2003, 2006, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Holger Hans Peter Freyther
  * Copyright (C) 2014 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 61 matching lines @@
     Vector<SVGGlyph::ArabicForm> m_arabicForms;
 #endif
 
 private:
     struct CharacterData {
         UChar32 character;
         unsigned clusterLength;
         int characterOffset;
     };
 
-    GlyphData glyphDataForCharacter(CharacterData&);
+    GlyphData glyphDataForCharacter(CharacterData&, bool spaceNormalize = false);

[eae, 2014/09/16 00:46:34]

normalizeSpace

[h.joshi, 2014/09/16 05:48:11]

Done.

     float characterWidth(UChar32, const GlyphData&) const;
-    void cacheFallbackFont(UChar32, const SimpleFontData*, const SimpleFontData* primaryFont);
+    void cacheFallbackFont(const SimpleFontData*, const SimpleFontData* primaryFont);

[eae, 2014/09/16 00:46:34]

Why is the character param no longer needed here?

[h.joshi, 2014/09/16 05:48:11]

"glyphDataAndPageForCharacter" method takes care of NormalVariant and SmallCaps
variant, so passing of "character" is not required now.
 This change is not exactly related to current "normalize space" changes, I was
making changes and need to pass "normalize space flag" to this method also. But
as "character" is not required in this method so modified calling parameters.

     float adjustSpacing(float, const CharacterData&, const SimpleFontData&, GlyphBuffer*);
     void updateGlyphBounds(const GlyphData&, float width, bool firstCharacter);
 
     template <typename TextIterator>
     unsigned advanceInternal(TextIterator&, GlyphBuffer*);
 
     HashSet<const SimpleFontData*>* m_fallbackFonts;
     float m_maxGlyphBoundingBoxY;
     float m_minGlyphBoundingBoxY;
     float m_firstGlyphOverflow;
     float m_lastGlyphOverflow;
 
     bool m_accountForGlyphBounds : 1;
     bool m_forTextEmphasis : 1;
 };
 
 } // namespace blink
 
 #endif