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

Unified Diff: cc/layers/scrollbar_layer_unittest.cc

Issue 2762123004: cc: LayerTreeHostImpl uses element id to tick animations (Closed)
Patch Set: review comments 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
Download patch
Index: cc/layers/scrollbar_layer_unittest.cc
diff --git a/cc/layers/scrollbar_layer_unittest.cc b/cc/layers/scrollbar_layer_unittest.cc
index f61fb42bfacf71b8784bb093ea8ccede0a1762e5..a429b0b3bc4fa8778692be9dc123ecffd055b38a 100644
--- a/cc/layers/scrollbar_layer_unittest.cc
+++ b/cc/layers/scrollbar_layer_unittest.cc
@@ -627,10 +627,10 @@ TEST_F(ScrollbarLayerTest, ScrollbarLayerOpacity) {
EXPECT_EQ(node->opacity, 0.f);
// This tests that activation does not change the opacity of scrollbar layer.
- LayerImpl* scrollbar_layer_impl =
- layer_tree_impl->LayerById(scrollbar_layer->id());
- layer_tree_impl->property_trees()->effect_tree.OnOpacityAnimated(
- 0.25f, scrollbar_layer_impl->effect_tree_index(), layer_tree_impl);
+ ScrollbarLayerImplBase* scrollbar_layer_impl =
+ static_cast<ScrollbarLayerImplBase*>(
+ layer_tree_impl->LayerById(scrollbar_layer->id()));
+ scrollbar_layer_impl->SetOverlayScrollbarLayerOpacityAnimated(0.25f);
host_impl->CreatePendingTree();
layer_impl_tree_root = layer_tree_host_->CommitAndCreatePendingTree();
layer_tree_impl = layer_impl_tree_root->layer_tree_impl();

Powered by Google App Engine
This is Rietveld 408576698