Bug 46193

Summary: [chromium] Composited layers get double transformed when scrolling
Product: WebKit Reporter: Vangelis Kokkevis <vangelis>
Component: WebCore Misc.Assignee: Vangelis Kokkevis <vangelis>
Status: RESOLVED FIXED    
Severity: Normal CC: jamesr, kbr, nduca
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
proposed patch kbr: review+

Description Vangelis Kokkevis 2010-09-21 09:42:54 PDT
There's a new scroll layer now maintained by the common RenderLayerCompositor which applies the scrolling transform.  The additional transform added by the LayerRendererChromium should now be removed.
Comment 1 Vangelis Kokkevis 2010-09-21 10:13:39 PDT
Created attachment 68255 [details]
proposed patch

updateLayersRecursive is (incorrectly) also called by LayerRendererChromium::prepareToDrawLayers() but the invocation in drawLayers() is the one that really matters.  I'll clean up the prepareToDrawLayers() in a separate CL.
Comment 2 Kenneth Russell 2010-09-21 10:44:28 PDT
Comment on attachment 68255 [details]
proposed patch

Looks fine to me.
Comment 3 Vangelis Kokkevis 2010-09-21 10:51:48 PDT
Committed r67966: <http://trac.webkit.org/changeset/67966>