Bug 165147

Summary: Address post-review feedback following r209070 (AppKit momentum scrolling adoption for scroll snapping)
Product: WebKit Reporter: Wenson Hsieh <wenson_hsieh>
Component: WebKit Misc.Assignee: Wenson Hsieh <wenson_hsieh>
Status: RESOLVED FIXED    
Severity: Normal CC: cmarcelo, commit-queue, jamesr, luiz, simon.fraser, tonikitoo, zalan
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Wenson Hsieh
Reported 2016-11-29 10:34:38 PST
See comments from Simon and Zalan in https://bugs.webkit.org/show_bug.cgi?id=147261
Attachments
Patch (34.50 KB, patch)
2016-11-29 11:37 PST, Wenson Hsieh 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Wenson Hsieh
Comment 1 2016-11-29 11:37:54 PST
Created attachment 295613 [details] Patch
alan
Comment 2 2016-11-29 12:28:06 PST
Comment on attachment 295613 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=295613&action=review > Source/WebCore/page/Page.cpp:2002 > + } This looks so much better now.
WebKit Commit Bot
Comment 3 2016-11-29 13:12:46 PST
Comment on attachment 295613 [details] Patch Clearing flags on attachment: 295613 Committed r209087: <http://trac.webkit.org/changeset/209087>
WebKit Commit Bot
Comment 4 2016-11-29 13:12:50 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.