According to the leak check run-safari does when quitting Safari, we're leaking 39 Nodes after running Acid3. Using debug build of r31371 on 10.5.2 (9C7010) with Safari 3.1 (5525.13). MallocStackLogging is turned on, opening Safari, running Acid3 (empty page set as homepage), closing the browser window and waiting 10 seconds, then running leaks on Safari via the command line. Cached/uncached load does not make a difference.
Created attachment 20126 [details] Leaks output
This might be related to the obvious leak of SimplFontData objects from: SimpleFontData* CSSFontFaceSource::getFontData(const FontDescription& fontDescription, bool syntheticBold, bool syntheticItalic, CSSFontSelector* fontSelector) That method is named "get" but we return new'd pointers in the SVG fonts case. Bad bad.
(In reply to comment #2) > This might be related to the obvious leak of SimplFontData objects from: > SimpleFontData* CSSFontFaceSource::getFontData(const FontDescription& > fontDescription, bool syntheticBold, bool syntheticItalic, CSSFontSelector* > fontSelector) > > That method is named "get" but we return new'd pointers in the SVG fonts case. > Bad bad. > Ok, I was wrong. These get stuffed off into m_fontDataTable and released when pruneTable() is called.