Bug 60677

Summary: Layout unnecessarily interrupts smooth scrolling at msnbc.com
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: mitz
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Avoid an unnecessary immediate scroll in updateScrollbars() darin: review+

mitz
Reported 2011-05-11 17:28:11 PDT
Layout unnecessarily interrupts smooth scrolling at msnbc.com
Attachments
Avoid an unnecessary immediate scroll in updateScrollbars() (1.91 KB, patch)
2011-05-11 17:32 PDT, mitz 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2011-05-11 17:32:14 PDT
Created attachment 93220 [details] Avoid an unnecessary immediate scroll in updateScrollbars()
Darin Adler
Comment 2 2011-05-11 17:37:23 PDT
Comment on attachment 93220 [details] Avoid an unnecessary immediate scroll in updateScrollbars() View in context: https://bugs.webkit.org/attachment.cgi?id=93220&action=review > Source/WebCore/platform/ScrollView.cpp:586 > + IntPoint scrollPoint = adjustScrollPositionWithinRange(IntPoint(desiredOffset)) + IntSize(m_scrollOrigin.x(), m_scrollOrigin.y()); Wish there was a cleaner way to convert a point to a size.
mitz
Comment 3 2011-05-11 17:46:23 PDT
Fixed in r86293. <http://trac.webkit.org/changeset/86293>
Note You need to log in before you can comment on or make changes to this bug.