Don't populate GlyphBuffer in InlineTextBox::characterWidths

Source/core/rendering/InlineTextBox.cpp

Index: Source/core/rendering/InlineTextBox.cpp
diff --git a/Source/core/rendering/InlineTextBox.cpp b/Source/core/rendering/InlineTextBox.cpp
index 32d4485c72c341d3c8d6579a8fff1393359becc7..eaae5cfa202fcd498d84c3d9075be26fa580d7ae 100644
--- a/Source/core/rendering/InlineTextBox.cpp
+++ b/Source/core/rendering/InlineTextBox.cpp
@@ -1237,12 +1237,11 @@ void InlineTextBox::characterWidths(Vector<float>& widths) const
 
     TextRun textRun = constructTextRun(styleToUse, font);
 
-    GlyphBuffer glyphBuffer;
     WidthIterator it(&font, textRun);
     float lastWidth = 0;
     widths.resize(m_len);
     for (unsigned i = 0; i < m_len; i++) {
-        it.advance(i + 1, &glyphBuffer);
+        it.advance(i + 1);

[Dominik Röttsches, 2014/09/29 08:06:55]

We can do it like this - but alternatively, it looks like we could replace the
characterWidths function to just return the complete advances-array of the glyph
buffer, now that we have the new representation of advances + offsets, couldn't
we? That might be better than doing a copy loop by character index here.

[eae, 2014/09/29 13:15:15]

Hmm, that might work. I'll look into it.

         widths[i] = it.m_runWidthSoFar - lastWidth;
         lastWidth = it.m_runWidthSoFar;
     }