cc/trees/layer_tree_host_unittest.cc - Issue 2762123004: cc: LayerTreeHostImpl uses element id to tick animations

Unified Diff: cc/trees/layer_tree_host_unittest.cc

Issue 2762123004: cc: LayerTreeHostImpl uses element id to tick animations (Closed)

Patch Set: remove accidental debug print Created 3 years, 9 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

Index: cc/trees/layer_tree_host_unittest.cc

diff --git a/cc/trees/layer_tree_host_unittest.cc b/cc/trees/layer_tree_host_unittest.cc

index fb3eaf5d0a422b4478f1f83327afe4ef8a0855e6..2e4756e29b1c3492c4ea07e5cc6231de8f97c940 100644

--- a/cc/trees/layer_tree_host_unittest.cc

+++ b/cc/trees/layer_tree_host_unittest.cc

@@ -1119,8 +1119,8 @@ class LayerTreeHostTestPropertyTreesChangedSync : public LayerTreeHostTest {

case OPACITY:

index_++;

impl->active_tree()->ResetAllChangeTracking();

- impl->active_tree()->property_trees()->effect_tree.OnOpacityAnimated(

- 0.5f, root->effect_tree_index(), impl->active_tree());

+ impl->active_tree()->SetTreeLayerOpacityMutated(root->element_id(),

+ 0.5f);

PostSetNeedsCommitToMainThread();

break;

case TRANSFORM:

@@ -1136,10 +1136,8 @@ class LayerTreeHostTestPropertyTreesChangedSync : public LayerTreeHostTest {

->LayerById(child_->id())

->LayerPropertyChanged());

transform.Translate(10, 10);

- impl->active_tree()

- ->property_trees()

- ->transform_tree.OnTransformAnimated(

- transform, root->transform_tree_index(), impl->active_tree());

+ impl->active_tree()->SetTreeLayerTransformMutated(root->element_id(),

+ transform);

PostSetNeedsCommitToMainThread();

break;

case FILTER:

@@ -1154,8 +1152,8 @@ class LayerTreeHostTestPropertyTreesChangedSync : public LayerTreeHostTest {

->LayerById(child_->id())

->LayerPropertyChanged());

filters.Append(FilterOperation::CreateOpacityFilter(0.5f));

- impl->active_tree()->property_trees()->effect_tree.OnFilterAnimated(

- filters, root->effect_tree_index(), impl->active_tree());

+ impl->active_tree()->SetTreeLayerFilterMutated(root->element_id(),

+ filters);

PostSetNeedsCommitToMainThread();

break;

case END:

@@ -1229,8 +1227,8 @@ class LayerTreeHostTestEffectTreeSync : public LayerTreeHostTest {

EffectNode* node = effect_tree.Node(root->effect_tree_index());

switch (impl->sync_tree()->source_frame_number()) {

case 0:

- effect_tree.OnOpacityAnimated(0.75f, root->effect_tree_index(),

- impl->sync_tree());

+ impl->sync_tree()->SetTreeLayerOpacityMutated(root->element_id(),

+ 0.75f);

PostSetNeedsCommitToMainThread();

break;

case 1:

@@ -1239,8 +1237,8 @@ class LayerTreeHostTestEffectTreeSync : public LayerTreeHostTest {

break;

case 2:

EXPECT_EQ(node->opacity, 0.75f);

- effect_tree.OnOpacityAnimated(0.75f, root->effect_tree_index(),

- impl->sync_tree());

+ impl->sync_tree()->SetTreeLayerOpacityMutated(root->element_id(),

+ 0.75f);

PostSetNeedsCommitToMainThread();

break;

case 3:

@@ -1249,8 +1247,8 @@ class LayerTreeHostTestEffectTreeSync : public LayerTreeHostTest {

break;

case 4:

EXPECT_EQ(node->opacity, 0.25f);

- impl->sync_tree()->property_trees()->effect_tree.OnFilterAnimated(

- brightness_filter_, root->effect_tree_index(), impl->sync_tree());

+ impl->sync_tree()->SetTreeLayerFilterMutated(root->element_id(),

+ brightness_filter_);

PostSetNeedsCommitToMainThread();

break;

case 5:

@@ -1259,8 +1257,8 @@ class LayerTreeHostTestEffectTreeSync : public LayerTreeHostTest {

break;

case 6:

EXPECT_EQ(node->filters, brightness_filter_);

- impl->sync_tree()->property_trees()->effect_tree.OnFilterAnimated(

- brightness_filter_, root->effect_tree_index(), impl->sync_tree());

+ impl->sync_tree()->SetTreeLayerFilterMutated(root->element_id(),

+ brightness_filter_);

PostSetNeedsCommitToMainThread();

break;

case 7:

@@ -1322,16 +1320,17 @@ class LayerTreeHostTestTransformTreeSync : public LayerTreeHostTest {

void CommitCompleteOnThread(LayerTreeHostImpl* impl) override {

TransformTree& transform_tree =

impl->sync_tree()->property_trees()->transform_tree;

- TransformNode* node = transform_tree.Node(

- impl->sync_tree()->root_layer_for_testing()->transform_tree_index());

+ const LayerImpl* root_layer = impl->sync_tree()->root_layer_for_testing();

+ const TransformNode* node =

+ transform_tree.FindNodeFromOwningLayerId(root_layer->id());

gfx::Transform rotate10;

rotate10.Rotate(10.f);

gfx::Transform rotate20;

rotate20.Rotate(20.f);

switch (impl->sync_tree()->source_frame_number()) {

case 0:

- impl->sync_tree()->property_trees()->transform_tree.OnTransformAnimated(

- rotate20, node->id, impl->sync_tree());

+ impl->sync_tree()->SetTreeLayerTransformMutated(

+ root_layer->element_id(), rotate20);

PostSetNeedsCommitToMainThread();

break;

case 1:

@@ -1340,8 +1339,8 @@ class LayerTreeHostTestTransformTreeSync : public LayerTreeHostTest {

break;

case 2:

EXPECT_EQ(node->local, rotate20);

- impl->sync_tree()->property_trees()->transform_tree.OnTransformAnimated(

- rotate20, node->id, impl->sync_tree());

+ impl->sync_tree()->SetTreeLayerTransformMutated(

+ root_layer->element_id(), rotate20);

PostSetNeedsCommitToMainThread();

break;

case 3:

@@ -1425,8 +1424,8 @@ class LayerTreeHostTestTransformTreeDamageIsUpdated : public LayerTreeHostTest {

if (impl->active_tree()->source_frame_number() == 0) {

gfx::Transform scale;

scale.Scale(2.0, 2.0);

- impl->active_tree()->property_trees()->transform_tree.OnTransformAnimated(

- scale, child_->transform_tree_index(), impl->active_tree());

+ impl->active_tree()->SetTreeLayerTransformMutated(child_->element_id(),

+ scale);

}

« cc/layers/scrollbar_layer_impl_base.cc ('K') | « cc/trees/layer_tree_host_impl.cc ('k') | cc/trees/layer_tree_host_unittest_picture.cc » ('j') | cc/trees/layer_tree_impl.h » ('J')