[webkit-reviews] review requested: [Bug 66438] Abandoned Memory: SVGFontElement and Corresponding SVGDocument Never Deconstructed : [Attachment 104298] [PATCH] Proposed Fix

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 17 19:11:24 PDT 2011


Joseph Pecoraro <joepeck at webkit.org> has asked	for review:
Bug 66438: Abandoned Memory: SVGFontElement and Corresponding SVGDocument Never
Deconstructed
https://bugs.webkit.org/show_bug.cgi?id=66438

Attachment 104298: [PATCH] Proposed Fix
https://bugs.webkit.org/attachment.cgi?id=104298&action=review

------- Additional Comments from Joseph Pecoraro <joepeck at webkit.org>
This fixes the problem and didn't cause any tests to crash. I can't think
of a scenario where someone would like to keep an SVGFontFace element
around without its corresponding SVGFontElement. Maybe we can add
some kind of ASSERTs to catch something like that early.


More information about the webkit-reviews mailing list