RESOLVED FIXED 115060
Don't create compositing layers for sticky position unless using the ScrollingCoordinator 
https://bugs.webkit.org/show_bug.cgi?id=115060
Summary Don't create compositing layers for sticky position unless using the Scrollin...
Simon Fraser (smfr)
Reported 2013-04-23 14:23:06 PDT
Don't create compositing layers for sticky position unless using the ScrollingCoordinator
Attachments
Patch (10.37 KB, patch)
2013-04-23 14:25 PDT, Simon Fraser (smfr) 		thorton: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2013-04-23 14:25:44 PDT
Created attachment 199325 [details] Patch
Tim Horton
Comment 2 2013-04-23 14:28:57 PDT
Comment on attachment 199325 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=199325&action=review > Source/WebCore/rendering/RenderLayerCompositor.cpp:2193 > + if (ScrollingCoordinator* scrollingCoordinator = this->scrollingCoordinator()) > + compositeSticky = scrollingCoordinator->coordinatesScrollingForFrameView(m_renderView->frameView()); > + return compositeSticky; The same logic exists in other places (shouldCompositeOverflowControls, for one). Should we maybe factor it out?
Simon Fraser (smfr)
Comment 3 2013-04-23 15:00:39 PDT
http://trac.webkit.org/changeset/148998
Note You need to log in before you can comment on or make changes to this bug.