[Webkit-unassigned] [Bug 117457] New: Remove the inline Vector capacity in SVGGlyphMap

[bugzilla-daemon at webkit.org]

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

           Summary: Remove the inline Vector capacity in SVGGlyphMap
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Keywords: BlinkMergeCandidate
          Severity: Normal
          Priority: P2
         Component: SVG
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: rniwa at webkit.org
                CC: zimmermann at kde.org, barraclough at apple.com,
                    pdr at google.com, thorton at apple.com


Consider merging https://chromium.googlesource.com/chromium/blink/+/f77a5392726f68494fc2bff8024b181549edca66

Don't pre-reserve massive amounts of memory in SVGGlyphMap.

This reduces the size of SVGFontElement from 20KB_ to ~200 bytes.

diff --git a/Source/core/svg/SVGGlyphMap.h b/Source/core/svg/SVGGlyphMap.h
index 1db5048..1e4153b 100644
--- a/Source/core/svg/SVGGlyphMap.h
+++ b/Source/core/svg/SVGGlyphMap.h
@@ -155,7 +155,7 @@

 private:
     GlyphMapLayer m_rootLayer;
-    Vector<SVGGlyph, 256> m_glyphTable;
+    Vector<SVGGlyph> m_glyphTable;
     HashMap<String, Glyph> m_namedGlyphs;
     int m_currentPriority;
 };

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