SimpleFontData::fillGlyphPage should not set zero for missing glyphs

Source/core/svg/SVGFontData.cpp

 /*
  * Copyright (C) 2008 Nikolas Zimmermann <zimmermann@kde.org>
  *
  * 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.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 238 matching lines @@
     return fillNonBMPGlyphs(fontElement, pageToFill, offset, length, buffer, fontData);
 }
 
 bool SVGFontData::fillBMPGlyphs(SVGFontElement* fontElement, GlyphPage* pageToFill, unsigned offset, unsigned length, UChar* buffer, const SimpleFontData* fontData) const
 {
     bool haveGlyphs = false;
     Vector<SVGGlyph> glyphs;
     for (unsigned i = 0; i < length; ++i) {
         String lookupString(buffer + i, 1);
         fontElement->collectGlyphsForString(lookupString, glyphs);
-        if (glyphs.isEmpty()) {
-            pageToFill->setGlyphDataForIndex(offset + i, 0, 0);
+        if (glyphs.isEmpty())
             continue;
-        }
 
         // Associate entry in glyph page with first valid SVGGlyph.
         // If there are multiple valid ones, just take the first one. WidthIterator will take
         // care of matching to the correct glyph, if multiple ones are available, as that's
         // only possible within the context of a string (eg. arabic form matching).
         haveGlyphs = true;
         pageToFill->setGlyphDataForIndex(offset + i, glyphs.first().tableEntry, fontData);
         glyphs.clear();
     }
 
     return haveGlyphs;
 }
 
 bool SVGFontData::fillNonBMPGlyphs(SVGFontElement* fontElement, GlyphPage* pageToFill, unsigned offset, unsigned length, UChar* buffer, const SimpleFontData* fontData) const
 {
     bool haveGlyphs = false;
     Vector<SVGGlyph> glyphs;
     for (unsigned i = 0; i < length; ++i) {
         // Each character here consists of a surrogate pair
         String lookupString(buffer + i * 2, 2);
         fontElement->collectGlyphsForString(lookupString, glyphs);
-        if (glyphs.isEmpty()) {
-            pageToFill->setGlyphDataForIndex(offset + i, 0, 0);
+        if (glyphs.isEmpty())
             continue;
-        }
 
         // Associate entry in glyph page with first valid SVGGlyph.
         // If there are multiple valid ones, just take the first one. WidthIterator will take
         // care of matching to the correct glyph, if multiple ones are available, as that's
         // only possible within the context of a string (eg. arabic form matching).
         haveGlyphs = true;
         pageToFill->setGlyphDataForIndex(offset + i, glyphs.first().tableEntry, fontData);
         glyphs.clear();
     }
 
@@ skipping 36 matching lines @@
 
 bool SVGFontData::shouldSkipDrawing() const
 {
     // FIXME: (http://crbug.com/359380) Glyph may be referenced after removeNode from the current editing impl.
     return !m_svgFontFaceElement || !m_svgFontFaceElement->inDocument();
 }
 
 } // namespace WebCore
 
 #endif