summaryrefslogtreecommitdiffstats
path: root/ui/compositor
diff options
context:
space:
mode:
authorloyso <loyso@chromium.org>2016-03-13 22:07:45 -0700
committerCommit bot <commit-bot@chromium.org>2016-03-14 05:08:44 +0000
commitdb73537f8ed057da02312224a39c92a3f29bf4e4 (patch)
tree508f45a39619418a66f49603b83d5e95ff447796 /ui/compositor
parentb28f153c67525946045e80ea4de9fec491d101ed (diff)
downloadchromium_src-db73537f8ed057da02312224a39c92a3f29bf4e4.zip
chromium_src-db73537f8ed057da02312224a39c92a3f29bf4e4.tar.gz
chromium_src-db73537f8ed057da02312224a39c92a3f29bf4e4.tar.bz2
CC Animation: Rename HasAnimation to HasAnimationForTesting.
Also, erase Layer::IsActive method (it is meaningless and not used anywhere) BUG=575041 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1797703002 Cr-Commit-Position: refs/heads/master@{#380926}
Diffstat (limited to 'ui/compositor')
-rw-r--r--ui/compositor/layer_animator_unittest.cc14
1 files changed, 10 insertions, 4 deletions
diff --git a/ui/compositor/layer_animator_unittest.cc b/ui/compositor/layer_animator_unittest.cc
index 9c1c5f1..b4afb64 100644
--- a/ui/compositor/layer_animator_unittest.cc
+++ b/ui/compositor/layer_animator_unittest.cc
@@ -2526,18 +2526,21 @@ TEST(LayerAnimatorTest, LayerMovedBetweenCompositorsDuringAnimation) {
Layer layer;
root_1.Add(&layer);
LayerAnimator* animator = layer.GetAnimator();
+ EXPECT_FALSE(layer.cc_layer_for_testing()->HasActiveAnimationForTesting());
+
double target_opacity = 1.0;
base::TimeDelta time_delta = base::TimeDelta::FromSeconds(1);
+
animator->ScheduleAnimation(new LayerAnimationSequence(
LayerAnimationElement::CreateOpacityElement(target_opacity, time_delta)));
EXPECT_TRUE(compositor_1->layer_animator_collection()->HasActiveAnimators());
EXPECT_FALSE(compositor_2->layer_animator_collection()->HasActiveAnimators());
- EXPECT_TRUE(layer.cc_layer_for_testing()->HasActiveAnimation());
+ EXPECT_TRUE(layer.cc_layer_for_testing()->HasActiveAnimationForTesting());
root_2.Add(&layer);
EXPECT_FALSE(compositor_1->layer_animator_collection()->HasActiveAnimators());
EXPECT_TRUE(compositor_2->layer_animator_collection()->HasActiveAnimators());
- EXPECT_TRUE(layer.cc_layer_for_testing()->HasActiveAnimation());
+ EXPECT_TRUE(layer.cc_layer_for_testing()->HasActiveAnimationForTesting());
host_2.reset();
host_1.reset();
@@ -2564,13 +2567,16 @@ TEST(LayerAnimatorTest, ThreadedAnimationSurvivesIfLayerRemovedAdded) {
LayerAnimator* animator = layer.GetAnimator();
double target_opacity = 1.0;
base::TimeDelta time_delta = base::TimeDelta::FromSeconds(1);
+
animator->ScheduleAnimation(new LayerAnimationSequence(
LayerAnimationElement::CreateOpacityElement(target_opacity, time_delta)));
+ EXPECT_TRUE(layer.cc_layer_for_testing()->HasActiveAnimationForTesting());
- EXPECT_TRUE(layer.cc_layer_for_testing()->HasActiveAnimation());
root.Remove(&layer);
+ EXPECT_FALSE(layer.cc_layer_for_testing()->HasActiveAnimationForTesting());
+
root.Add(&layer);
- EXPECT_TRUE(layer.cc_layer_for_testing()->HasActiveAnimation());
+ EXPECT_TRUE(layer.cc_layer_for_testing()->HasActiveAnimationForTesting());
host.reset();
TerminateContextFactoryForTests();