WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED CONFIGURATION CHANGED
117457
Remove the inline Vector capacity in SVGGlyphMap
https://bugs.webkit.org/show_bug.cgi?id=117457
Summary
Remove the inline Vector capacity in SVGGlyphMap
Ryosuke Niwa
Reported
2013-06-10 17:59:40 PDT
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; };
Attachments
Add attachment
proposed patch, testcase, etc.
Ahmad Saleem
Comment 1
2022-08-13 05:11:23 PDT
This commit removed "SVGGlyphMap.h":
https://github.com/WebKit/WebKit/commit/4738cfa45c28bf60411bd29119e154f4bae7d963
I am also not able to find anything with "GlyphMapLayer", is this needed anymore? Thanks!
Ryosuke Niwa
Comment 2
2022-08-13 10:44:18 PDT
We got rid of this code.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug