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

Unified Diff: ui/compositor/transform_animation_curve_adapter.cc

Issue 12035029: Finish migrating cc/ from WebKit::WebTransformationMatrix to gfx::Transform (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/compositor/transform_animation_curve_adapter.cc
diff --git a/ui/compositor/transform_animation_curve_adapter.cc b/ui/compositor/transform_animation_curve_adapter.cc
index 5700ffd41a198539c8b92f16d0ae140cdc0b4508..f7ad0f136b720a4e55a8dccd4bb50f6802fc1815 100644
--- a/ui/compositor/transform_animation_curve_adapter.cc
+++ b/ui/compositor/transform_animation_curve_adapter.cc
@@ -4,33 +4,6 @@
#include "ui/compositor/transform_animation_curve_adapter.h"
-using WebKit::WebTransformationMatrix;
-
-namespace {
-// TODO(ajuma): Remove this once the return type of
-// cc::TransformAnimationCurve::getValue is changed to gfx::Transform.
-WebTransformationMatrix convertTransformToWebTransformationMatrix(
- const gfx::Transform& transform) {
- return WebTransformationMatrix(transform.matrix().getDouble(0, 0),
- transform.matrix().getDouble(1, 0),
- transform.matrix().getDouble(2, 0),
- transform.matrix().getDouble(3, 0),
- transform.matrix().getDouble(0, 1),
- transform.matrix().getDouble(1, 1),
- transform.matrix().getDouble(2, 1),
- transform.matrix().getDouble(3, 1),
- transform.matrix().getDouble(0, 2),
- transform.matrix().getDouble(1, 2),
- transform.matrix().getDouble(2, 2),
- transform.matrix().getDouble(3, 2),
- transform.matrix().getDouble(0, 3),
- transform.matrix().getDouble(1, 3),
- transform.matrix().getDouble(2, 3),
- transform.matrix().getDouble(3, 3));
-
-}
-} // namespace
-
namespace ui {
TransformAnimationCurveAdapter::TransformAnimationCurveAdapter(
@@ -62,12 +35,12 @@ scoped_ptr<cc::AnimationCurve> TransformAnimationCurveAdapter::clone() const {
return to_return.PassAs<cc::AnimationCurve>();
}
-WebTransformationMatrix TransformAnimationCurveAdapter::getValue(
+gfx::Transform TransformAnimationCurveAdapter::getValue(
double t) const {
if (t >= duration_.InSecondsF())
- return convertTransformToWebTransformationMatrix(target_value_);
+ return target_value_;
if (t <= 0.0)
- return convertTransformToWebTransformationMatrix(initial_value_);
+ return initial_value_;
double progress = t / duration_.InSecondsF();
gfx::DecomposedTransform to_return;
@@ -75,8 +48,7 @@ WebTransformationMatrix TransformAnimationCurveAdapter::getValue(
decomposed_initial_value_,
decomposed_target_value_,
Tween::CalculateValue(tween_type_, progress));
- return convertTransformToWebTransformationMatrix(
- gfx::ComposeTransform(to_return));
+ return gfx::ComposeTransform(to_return);
}
} // namespace ui

Powered by Google App Engine
This is Rietveld 408576698