Summary: | Close the 19,000+ leaks introduced during the DOM merger | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||
Component: | SVG | Assignee: | Eric Seidel (no email) <eric> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | ||||||
Priority: | P4 | ||||||
Version: | 420+ | ||||||
Hardware: | Mac | ||||||
OS: | OS X 10.4 | ||||||
Attachments: |
|
Description
Eric Seidel (no email)
2005-11-26 15:10:54 PST
The patch also includes a (necessary) optimization to KCanvasContainer::paint() and (RenderPath::paint()) to not push a new KRenderingDeviceContext onto the context stack if one has already been pushed. Not adding this optimization was causing CGContextBeginTransparency layer to have kittens... I was never able to fully understand why, except that the transparency layer methods, as part of their silent context swapping, seem to assume a certain CGContextRef, CGContextRelease pattern which we don't seem to follow, even though my previous code seemed perfectly legal. Created attachment 4811 [details]
Closes leaks from the DOM merger, adds back resource dumping, etc.
Comment on attachment 4811 [details]
Closes leaks from the DOM merger, adds back resource dumping, etc.
__sharedRenderingDevice
Please remove the underscores from this, identifiers w/ two leading underscores
are reserved to the implementation.
Other than that r=me
|