[Webkit-unassigned] [Bug 47030] Clean up resources when LayerRenderer changes

[bugzilla-daemon at webkit.org]

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





--- Comment #6 from Victoria Kirst <vrk at google.com>  2010-10-05 18:22:08 PST ---
(In reply to comment #3)
> I'd let her checkin first and I'll merge later.
Alpha actually already committed his linked patch (http://trac.webkit.org/changeset/69163), so my patch here has been modified (slightly) to fit with the current code.


(In reply to comment #4)
> (From update of attachment 69545 [details])
> This is copy-paste from ContentLayerChromium so we should move this logic up to the base class.  Specifically, add a virtual cleanupResources() function to LayerChromium and have LayerChromium::setLayerRenderer() invoke it.

Thanks for the tip! I added this and updated the related files. Take another look?

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