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

Unified Diff: ui/compositor/transform_animation_curve_adapter_unittest.cc

Issue 22861008: Add support for inverse transform animations. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix the DCHECK production code and removed k prefix to const transform variable. Created 7 years, 3 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_unittest.cc
diff --git a/ui/compositor/transform_animation_curve_adapter_unittest.cc b/ui/compositor/transform_animation_curve_adapter_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..613cf4af8c6684f6b4a2d0220b62b44a49a47067
--- /dev/null
+++ b/ui/compositor/transform_animation_curve_adapter_unittest.cc
@@ -0,0 +1,59 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "ui/compositor/transform_animation_curve_adapter.h"
+
+#include <sstream>
+
+#include "base/time/time.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "ui/compositor/test/test_utils.h"
+
+namespace ui {
+
+namespace {
+
+// Check that the inverse transform curve gives the gives a transform that when
+// applied on top of the parent transform gives the original transform
+TEST(InverseTransformCurveAdapterTest, InversesTransform) {
+ gfx::Transform parent_start, parent_target;
+ parent_start.Scale(0.5, 3.0);
+ parent_start.Translate(-20.0, 30.0);
+ parent_target.Translate(0, 100);
+
+ gfx::Transform child_transform;
+ child_transform.Rotate(-30.0);
+
+ base::TimeDelta duration = base::TimeDelta::FromSeconds(1);
+
+ const gfx::Transform effective_child_transform =
+ parent_start * child_transform;
+
+ TransformAnimationCurveAdapter parent_curve(Tween::LINEAR,
+ parent_start,
+ parent_target,
+ duration);
+
+ InverseTransformCurveAdapter child_curve(parent_curve,
+ child_transform,
+ duration);
+ static const int kSteps = 1000;
+ double step = 1.0 / kSteps;
+ for (int i = 0; i != kSteps + 1; ++i) {
danakj 2013/09/04 15:29:09 nit: i < kSteps + 1 ?
avallee 2013/09/04 15:36:53 I want to exit the loop having done 1001 steps in
danakj 2013/09/04 15:47:40 I'm not sure how < and != differ in this case. The
danakj 2013/09/04 15:48:26 (i <= kSteps) would also work if you preferred.
avallee 2013/09/04 15:50:34 Done.
+ std::ostringstream message;
+ message << "Step " << i << " of " << kSteps;
+ SCOPED_TRACE(message.str());
+ gfx::Transform progress_parent_transform =
+ parent_curve.GetValue(i*step);
+ gfx::Transform progress_child_transform =
+ child_curve.GetValue(i*step);
+ CheckApproximatelyEqual(effective_child_transform,
+ progress_parent_transform *
+ progress_child_transform);
+ }
+}
+
+} // namespace
+
+} // namespace ui
« ui/compositor/test/test_utils.cc ('K') | « ui/compositor/transform_animation_curve_adapter.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698