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

Unified Diff: cc/trees/layer_tree_host_common_unittest.cc

Issue 2881653002: Remove Layer{Impl}::TransformIsAnimating. (Closed)
Patch Set: Sync to head. Created 3 years, 7 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/layers/layer_impl.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/layer_tree_host_common_unittest.cc
diff --git a/cc/trees/layer_tree_host_common_unittest.cc b/cc/trees/layer_tree_host_common_unittest.cc
index e4a6f57d3d250c7efd054050ca9c8e0e2e64b671..aed6857c7a5666ec089d57058569c8ef53319777 100644
--- a/cc/trees/layer_tree_host_common_unittest.cc
+++ b/cc/trees/layer_tree_host_common_unittest.cc
@@ -2134,6 +2134,11 @@ TEST_F(LayerTreeHostCommonTest, LargeTransforms) {
EXPECT_EQ(gfx::Rect(), grand_child->visible_layer_rect());
}
+static bool TransformIsAnimating(LayerImpl* layer) {
+ return layer->GetMutatorHost()->IsAnimatingTransformProperty(
+ layer->element_id(), layer->GetElementTypeForAnimation());
+}
+
TEST_F(LayerTreeHostCommonTest,
ScreenSpaceTransformIsAnimatingWithDelayedAnimation) {
LayerImpl* root = root_layer_for_testing();
@@ -2166,7 +2171,7 @@ TEST_F(LayerTreeHostCommonTest,
EXPECT_FALSE(root->screen_space_transform_is_animating());
EXPECT_FALSE(child->screen_space_transform_is_animating());
- EXPECT_FALSE(grand_child->TransformIsAnimating());
+ EXPECT_FALSE(TransformIsAnimating(grand_child));
EXPECT_TRUE(grand_child->HasPotentiallyRunningTransformAnimation());
EXPECT_TRUE(grand_child->screen_space_transform_is_animating());
EXPECT_TRUE(great_grand_child->screen_space_transform_is_animating());
« no previous file with comments | « cc/layers/layer_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698