Bug 113057

Summary: [chromium] Allow scrolling past content edge on main-frame to account for scaled scrollbars.
Product: WebKit Reporter: W. James MacLean <wjmaclean>
Component: New BugsAssignee: W. James MacLean <wjmaclean>
Status: RESOLVED WONTFIX    
Severity: Normal CC: aelias, schenney
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch benjamin: review-

W. James MacLean
Reported 2013-03-22 07:40:45 PDT
[chromium] Allow scrolling past content edge on main-frame to account for scaled scrollbars.
Attachments
Patch (2.01 KB, patch)
2013-03-22 07:46 PDT, W. James MacLean 		benjamin: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
W. James MacLean
Comment 1 2013-03-22 07:46:53 PDT
Created attachment 194541 [details] Patch
W. James MacLean
Comment 2 2013-03-22 07:50:06 PDT
(In reply to comment #1) > Created an attachment (id=194541) [details] > Patch See https://code.google.com/p/chromium/issues/detail?id=223097
Alexandre Elias
Comment 3 2013-03-22 09:35:59 PDT
It looks like you intend to disable WebCore's scroll offset clamping entirely and do your own clamping. This change doesn't really do that since there are other places that modify the scroll position and go through the normal FrameView clamp: you'll run into more subtle bugs later. It sounds like the issue you're trying to fix with this is that the size of the scrollbars is computed incorrectly in ScrollView::visibleContentRect. So instead, I suggest fixing ScrollView's clamping logic to deal with that correctly. ScrollView::visibleContentScaleFactor() will return pageScaleFactor for the main frame and 1 for subframes, so it should be what you need.
Note You need to log in before you can comment on or make changes to this bug.