WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 81713
[chromium] Convert TouchFlingPlatformGestureCurve to a 2-D Bezier for better control of curve shape.
https://bugs.webkit.org/show_bug.cgi?id=81713
Summary
[chromium] Convert TouchFlingPlatformGestureCurve to a 2-D Bezier for better ...
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
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug