Bug 76196

Summary: Move snapRubberBandTimerFired to ScrollElasticityController
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch kling: review+

Description Anders Carlsson 2012-01-12 11:57:22 PST
Move snapRubberBandTimerFired to ScrollElasticityController
Comment 1 Anders Carlsson 2012-01-12 11:57:52 PST
Created attachment 122279 [details]
Patch
Comment 2 Andreas Kling 2012-01-12 12:02:12 PST
Comment on attachment 122279 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=122279&action=review

> Source/WebCore/ChangeLog:8
> +        No new tests. (OOPS!)

OOPS!

> Source/WebCore/platform/mac/ScrollElasticityController.mm:95
> +    if (!m_momentumScrollInProgress || m_ignoreMomentumScrolls) {

We could reverse this condition and do an early return to reduce nesting.
Comment 3 Anders Carlsson 2012-01-12 12:04:40 PST
Committed r104842: <http://trac.webkit.org/changeset/104842>