Bug 74817

Summary: The scrolling coordinator should know about the main frame scroll layer
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: levin+threading, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch kling: review+

Anders Carlsson
Reported 2011-12-18 12:15:06 PST
The scrolling coordinator should know about the main frame scroll layer
Attachments
Patch (4.53 KB, patch)
2011-12-18 12:16 PST, Anders Carlsson 		kling: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2011-12-18 12:16:36 PST
Created attachment 119773 [details] Patch
Andreas Kling
Comment 2 2011-12-18 12:23:39 PST
Comment on attachment 119773 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=119773&action=review > Source/WebCore/page/ScrollingCoordinator.h:60 > + // Should be called whenever the scroll layer for the given frame changes. > + void setFrameScrollLayer(Frame*, const GraphicsLayer* scrollLayer); > + > // Should be called whenever the geometry of the given frame changes, > // including the visible content rect and the content size. > void syncFrameGeometry(Frame*); frameScrollLayerChanged() and frameGeometryChanged() might be better names for these functions.
Anders Carlsson
Comment 3 2011-12-18 12:27:53 PST
Committed r103185: <http://trac.webkit.org/changeset/103185>
Note You need to log in before you can comment on or make changes to this bug.