Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(347)

Issue 11359077: cc: Replace the WebCore::UnitBezier class with the SkUnitCubicInterp() method from Skia. (Closed)

Created:
8 years, 1 month ago by danakj
Modified:
8 years, 1 month ago
Reviewers:
jamesr
CC:
chromium-reviews, cc-bugs_chromium.org, backer, piman
Visibility:
Public.

Description

cc: Replace the WebCore::UnitBezier class with the SkUnitCubicInterp() method from Skia. This method exists in Skia, but is not publicly consumable/linkable with a component build. Skia will expose this (or a similar) method through a proper public API, and we should use it. But in the meantime, we make a copy of the method in timing_function.cc, and use it there in place of WebCore's UnitBezier class. Tests: cc_unittests:TimingFunctionTest.CubicBezierTimingFunction This test compares the output of the timing function against baseline values recorded with WebCore's UnitBezier class. If new methods are able to come closer to those values, we should decrease the epsilon used in the test accordingly. R=jamesr BUG=147395 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=166362

Patch Set 1 #

Patch Set 2 : Fix Keyframe test #

Patch Set 3 : rebase #

Total comments: 2

Patch Set 4 : Add bug #

Patch Set 5 : Fix web unittests #

Unified diffs Side-by-side diffs Delta from patch set Stats (+127 lines, -35 lines) Patch
M cc/DEPS View 1 chunk +0 lines, -3 lines 0 comments Download
M cc/cc.gyp View 1 2 1 chunk +0 lines, -2 lines 0 comments Download
M cc/cc_tests.gyp View 1 2 1 chunk +1 line, -0 lines 0 comments Download
M cc/keyframed_animation_curve_unittest.cc View 1 1 chunk +1 line, -1 line 0 comments Download
D cc/stubs/UnitBezier.h View 1 chunk +0 lines, -6 lines 0 comments Download
D cc/stubs/unit_bezier.h View 1 chunk +0 lines, -14 lines 0 comments Download
M cc/timing_function.h View 2 chunks +5 lines, -2 lines 0 comments Download
M cc/timing_function.cc View 1 2 3 3 chunks +77 lines, -5 lines 0 comments Download
A cc/timing_function_unittest.cc View 1 chunk +41 lines, -0 lines 0 comments Download
M webkit/compositor_bindings/web_float_animation_curve_unittest.cc View 1 2 3 4 1 chunk +1 line, -1 line 0 comments Download
M webkit/compositor_bindings/web_transform_animation_curve_unittest.cc View 1 2 3 4 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
danakj
8 years, 1 month ago (2012-11-06 22:43:23 UTC) #1
jamesr
lgtm https://codereview.chromium.org/11359077/diff/5002/cc/timing_function.cc File cc/timing_function.cc (right): https://codereview.chromium.org/11359077/diff/5002/cc/timing_function.cc#newcode11 cc/timing_function.cc:11: // is available in the public Skia API, ...
8 years, 1 month ago (2012-11-06 23:21:19 UTC) #2
danakj
Thanks https://codereview.chromium.org/11359077/diff/5002/cc/timing_function.cc File cc/timing_function.cc (right): https://codereview.chromium.org/11359077/diff/5002/cc/timing_function.cc#newcode11 cc/timing_function.cc:11: // is available in the public Skia API, ...
8 years, 1 month ago (2012-11-06 23:26:51 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/danakj@chromium.org/11359077/2017
8 years, 1 month ago (2012-11-07 01:01:44 UTC) #4
commit-bot: I haz the power
8 years, 1 month ago (2012-11-07 04:59:35 UTC) #5
Change committed as 166362

Powered by Google App Engine
This is Rietveld 408576698