Bug 60677 - Layout unnecessarily interrupts smooth scrolling at msnbc.com
Summary: Layout unnecessarily interrupts smooth scrolling at msnbc.com
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: mitz
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-11 17:28 PDT by mitz
Modified: 2011-05-11 17:46 PDT (History)
0 users

See Also:

Attachments
Avoid an unnecessary immediate scroll in updateScrollbars() (1.91 KB, patch)
2011-05-11 17:32 PDT, mitz 		darin: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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>