[webkit-reviews] review granted: [Bug 83567] [BlackBerry] Crash when tearing down web page rendered using WebPageCompositor : [Attachment 136435] Patch

[bugzilla-daemon at webkit.org]

Antonio Gomes <tonikitoo at webkit.org> has granted Arvid Nilsson
<anilsson at rim.com>'s request for review:
Bug 83567: [BlackBerry] Crash when tearing down web page rendered using
WebPageCompositor
https://bugs.webkit.org/show_bug.cgi?id=83567

Attachment 136435: Patch
https://bugs.webkit.org/attachment.cgi?id=136435&action=review

------- Additional Comments from Antonio Gomes <tonikitoo at webkit.org>
(In reply to comment #3)
> (In reply to comment #2)
> > (From update of attachment 136435 [details] [details])
> > so if the mlayerRenderer is null, "resources" were already released?
> 
> Yes, or you could see it this way - the LayerRenderer owns the resources that
should be released, so if there's no LayerRenderer, there are no resources to
delete. Resources = textures right now, but could be more stuff in the future
(for example shaders).

Ok. I was wondering was would be the codepath be...