RESOLVED WONTFIX 66995
[chromium] Eradicate unneded RefPtr usage in compositor 
https://bugs.webkit.org/show_bug.cgi?id=66995
Summary [chromium] Eradicate unneded RefPtr usage in compositor
Nat Duca
Reported 2011-08-25 16:17:48 PDT
CCLayerTreeHost does not need to be refcounted. Nor does LayerRendererChromium. CCLayerImpl, I don't know? RenderSurfaces? Not if we go to a RenderPass abstraction and let the LayerRenderer manage the RenderPass list. Might be less fragile, anyway.
Attachments
Add attachment proposed patch, testcase, etc.
James Robinson
Comment 1 2011-09-02 18:11:47 PDT
I'm going to take a crack at CCLayerTreeHost/LayerRendererChromium.
James Robinson
Comment 2 2011-09-14 15:34:02 PDT
https://bugs.webkit.org/show_bug.cgi?id=68121 fixes LayerRendererChromium to not be RefCounted. CCLayerImpl is a bit tricky since currently the TreeSynchronizer relies on the refcounting. We can probably do this another way but it's not trivial. RenderSurfaceChromium and CCRenderSurface are single ownership (and need a rewrite anyway). CCLayerTreeHost is definitely possible, we just need to be careful to clear back pointers since the GraphicsLayers and LayerChromiums can very easily outlive the CCLayerTreeHost. I'll take a crack at this one next. After that I think we're pretty much done.
Note You need to log in before you can comment on or make changes to this bug.