SimpleFontData::fillGlyphPage should not set zero for missing glyphs

Source/platform/fonts/skia/SimpleFontDataSkia.cpp

 /*
  * Copyright (c) 2008, 2009, 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 275 matching lines @@
         SkDebugf("%s last char is high-surrogate", __FUNCTION__);
         return false;
     }
 
     SkAutoSTMalloc<GlyphPage::size, uint16_t> glyphStorage(length);
 
     uint16_t* glyphs = glyphStorage.get();
     SkTypeface* typeface = platformData().typeface();
     typeface->charsToGlyphs(buffer, SkTypeface::kUTF16_Encoding, glyphs, length);
 
-    unsigned allGlyphs = 0; // track if any of the glyphIDs are non-zero
+    bool haveGlyphs = false;
     for (unsigned i = 0; i < length; i++) {
-        pageToFill->setGlyphDataForIndex(offset + i, glyphs[i], glyphs[i] ? this : 0);
-        allGlyphs |= glyphs[i];
+        if (glyphs[i]) {
+            pageToFill->setGlyphDataForIndex(offset + i, glyphs[i], this);
+            haveGlyphs = true;
+        }
     }
 
-    return allGlyphs;
+    return haveGlyphs;
 }
 
 } // namespace WebCore