[Webkit-unassigned] [Bug 38530] SVG fonts may create blank Glyph Cache Pages

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=38530


mitz at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #55029|review?                     |review-
               Flag|                            |




--- Comment #2 from mitz at webkit.org  2010-05-04 11:25:37 PST ---
(From update of attachment 55029)
I don’t think this is the best fix. SegmentedFontData::isSVGFont()—and by
extension FontData::isSVGFont()— doesn’t make sense. Even though currently SVG
segments aren’t handled correctly due to bug 32227, introducing this concept is
a step in the wrong direction. Instead, GlyphPageTreeNode::initializePage()
should examine individual SimpleFontData for SVG-ness, and in the SVG case,
instead of calling fill() it should zero-fill and behave as if fill() returned
false. Since fill() has two call sites in initializePage(), you can add a small
static helper method that does this.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.