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

Unified Diff: cc/animation/keyframed_animation_curve_unittest.cc

Issue 2388943002: cc: Delete unused AnimationCurve::AffectsScale and related code (Closed)
Patch Set: Created 4 years, 2 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 | « cc/animation/keyframed_animation_curve.cc ('k') | cc/animation/transform_operations.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/animation/keyframed_animation_curve_unittest.cc
diff --git a/cc/animation/keyframed_animation_curve_unittest.cc b/cc/animation/keyframed_animation_curve_unittest.cc
index 520dad833ea324f6ea22955eee16cff1084fd0d4..e3f765939a763920e84be0e1500252f462782227 100644
--- a/cc/animation/keyframed_animation_curve_unittest.cc
+++ b/cc/animation/keyframed_animation_curve_unittest.cc
@@ -561,37 +561,6 @@ TEST(KeyframedAnimationCurveTest, AnimatedBounds) {
bounds.ToString());
}
-// Tests that animations that affect scale are correctly identified.
-TEST(KeyframedAnimationCurveTest, AffectsScale) {
- std::unique_ptr<KeyframedTransformAnimationCurve> curve(
- KeyframedTransformAnimationCurve::Create());
-
- TransformOperations operations1;
- curve->AddKeyframe(
- TransformKeyframe::Create(base::TimeDelta(), operations1, nullptr));
- operations1.AppendTranslate(2.0, 3.0, -1.0);
- TransformOperations operations2;
- operations2.AppendTranslate(4.0, 1.0, 2.0);
- curve->AddKeyframe(TransformKeyframe::Create(
- base::TimeDelta::FromSecondsD(1.f), operations2, nullptr));
-
- EXPECT_FALSE(curve->AffectsScale());
-
- TransformOperations operations3;
- operations3.AppendScale(2.f, 2.f, 2.f);
- curve->AddKeyframe(TransformKeyframe::Create(
- base::TimeDelta::FromSecondsD(2.f), operations3, nullptr));
-
- EXPECT_TRUE(curve->AffectsScale());
-
- TransformOperations operations4;
- operations3.AppendTranslate(2.f, 2.f, 2.f);
- curve->AddKeyframe(TransformKeyframe::Create(
- base::TimeDelta::FromSecondsD(3.f), operations4, nullptr));
-
- EXPECT_TRUE(curve->AffectsScale());
-}
-
// Tests that animations that are translations are correctly identified.
TEST(KeyframedAnimationCurveTest, IsTranslation) {
std::unique_ptr<KeyframedTransformAnimationCurve> curve(
« no previous file with comments | « cc/animation/keyframed_animation_curve.cc ('k') | cc/animation/transform_operations.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698