Always initialize |m_totalWidth| in HarfBuzzShaper::shape.

Source/platform/fonts/shaping/HarfBuzzShaper.cpp

Index: Source/platform/fonts/shaping/HarfBuzzShaper.cpp
diff --git a/Source/platform/fonts/shaping/HarfBuzzShaper.cpp b/Source/platform/fonts/shaping/HarfBuzzShaper.cpp
index 5a8d259d135af7f7dc8974cb8232cf9c2d9bbded..eec3af9436aa107a6c25c393621f8498b3cc246b 100644
--- a/Source/platform/fonts/shaping/HarfBuzzShaper.cpp
+++ b/Source/platform/fonts/shaping/HarfBuzzShaper.cpp
@@ -385,6 +385,7 @@ HarfBuzzShaper::HarfBuzzShaper(const Font* font, const TextRun& run, const Glyph
     , m_expansionOpportunityCount(0)
     , m_fromIndex(0)
     , m_toIndex(m_run.length())
+    , m_totalWidth(0)
 {
     m_normalizedBuffer = adoptArrayPtr(new UChar[m_run.length() + 1]);
     normalizeCharacters(m_run, m_run.length(), m_normalizedBuffer.get(), &m_normalizedBufferLength);
@@ -538,10 +539,10 @@ void HarfBuzzShaper::setFontFeatures()
 
 bool HarfBuzzShaper::shape(GlyphBuffer* glyphBuffer)
 {
+    m_totalWidth = 0;

[leviw_travelin_and_unemployed, 2015/04/27 18:19:35]

This should be redundant now that it's in the c'tor, no?

     if (!createHarfBuzzRuns())
         return false;
 
-    m_totalWidth = 0;
     if (!shapeHarfBuzzRuns())
         return false;