Bug 81713

Summary: [chromium] Convert TouchFlingPlatformGestureCurve to a 2-D Bezier for better control of curve shape.
Product: WebKit Reporter: W. James MacLean <wjmaclean>
Component: New BugsAssignee: W. James MacLean <wjmaclean>
Status: RESOLVED FIXED    
Severity: Normal CC: davemoore, enne, jamesr, rjkroege, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

W. James MacLean
Reported 2012-03-20 16:14:48 PDT
[chromium] Convert TouchFlingPlatformGestureCurve to a 2-D Bezier for better control of curve shape.
Attachments
Patch (5.63 KB, patch)
2012-03-20 16:23 PDT, W. James MacLean 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
W. James MacLean
Comment 1 2012-03-20 16:23:22 PDT
Created attachment 132924 [details] Patch
W. James MacLean
Comment 2 2012-03-20 16:24:20 PDT
(In reply to comment #1) > Created an attachment (id=132924) [details] > Patch The patch to rename this class will follow (maybe tomorrow morning at this point).
Nat Duca
Comment 3 2012-03-20 18:18:13 PDT
Comment on attachment 132924 [details] Patch Unofficially LGTM. Does @davemoore <3 this?
Dirk Pranke
Comment 4 2012-03-20 18:20:29 PDT
Comment on attachment 132924 [details] Patch stylistically the patch looks fine; normally I wouldn't touch a patch in this area with a ten foot pole, but nduca says it looks good ...
W. James MacLean
Comment 5 2012-03-20 18:23:45 PDT
(In reply to comment #3) > (From update of attachment 132924 [details]) > Unofficially LGTM. Does @davemoore <3 this? Yes, Dave has seen and thought it was good (not to speak for him of course!). Rob and Alex and I are happier with it than any of the other candidates - it feels pretty good.
WebKit Review Bot
Comment 6 2012-03-20 19:06:28 PDT
Comment on attachment 132924 [details] Patch Clearing flags on attachment: 132924 Committed r111483: <http://trac.webkit.org/changeset/111483>
WebKit Review Bot
Comment 7 2012-03-20 19:06:32 PDT
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.