WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
76196
Move snapRubberBandTimerFired to ScrollElasticityController
https://bugs.webkit.org/show_bug.cgi?id=76196
Summary
Move snapRubberBandTimerFired to ScrollElasticityController
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+
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug