Refactor some code that is used to check whether a layer needs backing store
Created attachment 131478 [details] Patch
Comment on attachment 131478 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=131478&action=review > Source/WebCore/rendering/RenderLayerBacking.cpp:-836 > - > - // If we have got this far and the renderer has no children, then we're ok. > - if (!renderObject->firstChild()) > - return true; It seems like this is no longer tested for an early return?
(In reply to comment #2) > (From update of attachment 131478 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=131478&action=review > > > Source/WebCore/rendering/RenderLayerBacking.cpp:-836 > > - > > - // If we have got this far and the renderer has no children, then we're ok. > > - if (!renderObject->firstChild()) > > - return true; The checks for child renderers and child layers effectively do the same test. > It seems like this is no longer tested for an early return?
http://trac.webkit.org/changeset/110601