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+

Anders Carlsson
Reported 2012-01-12 11:57:22 PST
Move snapRubberBandTimerFired to ScrollElasticityController
Attachments
Patch (11.86 KB, patch)
2012-01-12 11:57 PST, Anders Carlsson 		kling: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2012-01-12 11:57:52 PST
Created attachment 122279 [details] Patch
Andreas Kling
Comment 2 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.
Anders Carlsson
Comment 3 2012-01-12 12:04:40 PST
Committed r104842: <http://trac.webkit.org/changeset/104842>
Note You need to log in before you can comment on or make changes to this bug.