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+

Description mitz 2011-05-11 17:28:11 PDT 
Layout unnecessarily interrupts smooth scrolling at msnbc.com
Comment 1 mitz 2011-05-11 17:32:14 PDT 
Created attachment 93220 [details]
Avoid an unnecessary immediate scroll in updateScrollbars()
Comment 2 Darin Adler 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.
Comment 3 mitz 2011-05-11 17:46:23 PDT 
Fixed in r86293. <http://trac.webkit.org/changeset/86293>