[Webkit-unassigned] [Bug 36127] New: purgeInactiveFontData() should also prune glyph data

[bugzilla-daemon at webkit.org]

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

           Summary: purgeInactiveFontData() should also prune glyph data
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Text
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: yong.li.webkit at gmail.com
                CC: staikos at kde.org


FontCache::purgeInactiveFontData() is called to delete some inactive
SimpleFontData objects. But the deleted SimpleFontData objects' glyph data are
still left in glyph page tree. I think this can be considered as leaks.
Following code fixes it.

-    for (size_t i = 0; i < fontDataToDeleteCount; ++i)
+    for (size_t i = 0; i < fontDataToDeleteCount; ++i) {
+        GlyphPageTreeNode::pruneTreeFontData(fontDataToDelete[i]);
         delete fontDataToDelete[i];
+    }

Patch is following

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