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

Unified Diff: trunk/src/ui/compositor/transform_animation_curve_adapter_unittest.cc

Issue 23809002: Revert 220479 "Add support for inverse transform animations." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 4 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
« no previous file with comments | « trunk/src/ui/compositor/transform_animation_curve_adapter.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/ui/compositor/transform_animation_curve_adapter_unittest.cc
===================================================================
--- trunk/src/ui/compositor/transform_animation_curve_adapter_unittest.cc (revision 220491)
+++ trunk/src/ui/compositor/transform_animation_curve_adapter_unittest.cc (working copy)
@@ -1,59 +0,0 @@
-// 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 kEffective_child_transform =
- parent_start * child_transform;
-
- TransformAnimationCurveAdapter parent_curve(Tween::LINEAR,
- parent_start,
- parent_target,
- duration);
-
- InverseTransformCurveAdapter child_curve(parent_curve,
- child_transform,
- duration);
- const int kSteps = 1000;
- double step = 1.0 / kSteps;
- for (int i = 0; i != kSteps + 1; ++i) {
- 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(kEffective_child_transform,
- progress_parent_transform *
- progress_child_transform);
- }
-}
-
-} // namespace
-
-} // namespace ui
« no previous file with comments | « trunk/src/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