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

Side by Side Diff: cc/trees/layer_tree_host_unittest.cc

Issue 2105173006: cc: Clear entries from animations maps only when main thread wins (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove_DCHECK Created 4 years, 5 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 unified diff | Download patch
« no previous file with comments | « cc/layers/layer_impl.cc ('k') | cc/trees/layer_tree_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 The Chromium Authors. All rights reserved. 1 // Copyright 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/trees/layer_tree_host.h" 5 #include "cc/trees/layer_tree_host.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 982 matching lines...) Expand 10 before | Expand all | Expand 10 after
993 993
994 void DidCommit() override { 994 void DidCommit() override {
995 EffectTree& effect_tree = layer_tree_host()->property_trees()->effect_tree; 995 EffectTree& effect_tree = layer_tree_host()->property_trees()->effect_tree;
996 EffectNode* node = effect_tree.Node(root_->effect_tree_index()); 996 EffectNode* node = effect_tree.Node(root_->effect_tree_index());
997 switch (layer_tree_host()->source_frame_number()) { 997 switch (layer_tree_host()->source_frame_number()) {
998 case 1: 998 case 1:
999 node->opacity = 0.5f; 999 node->opacity = 0.5f;
1000 node->is_currently_animating_opacity = true; 1000 node->is_currently_animating_opacity = true;
1001 break; 1001 break;
1002 case 2: 1002 case 2:
1003 node->is_currently_animating_opacity = true;
1004 break;
1005 case 3:
1003 node->is_currently_animating_opacity = false; 1006 node->is_currently_animating_opacity = false;
1004 break; 1007 break;
1008 case 4:
1009 node->opacity = 0.25f;
1010 node->is_currently_animating_opacity = true;
1005 } 1011 }
1006 } 1012 }
1007 1013
1008 void CommitCompleteOnThread(LayerTreeHostImpl* impl) override { 1014 void CommitCompleteOnThread(LayerTreeHostImpl* impl) override {
1009 EffectTree& effect_tree = impl->sync_tree()->property_trees()->effect_tree; 1015 EffectTree& effect_tree = impl->sync_tree()->property_trees()->effect_tree;
1010 EffectNode* node = effect_tree.Node( 1016 EffectNode* node = effect_tree.Node(
1011 impl->sync_tree()->root_layer_for_testing()->effect_tree_index()); 1017 impl->sync_tree()->root_layer_for_testing()->effect_tree_index());
1012 switch (impl->sync_tree()->source_frame_number()) { 1018 switch (impl->sync_tree()->source_frame_number()) {
1013 case 0: 1019 case 0:
1014 impl->sync_tree()->root_layer_for_testing()->OnOpacityAnimated(0.75f); 1020 impl->sync_tree()->root_layer_for_testing()->OnOpacityAnimated(0.75f);
1015 PostSetNeedsCommitToMainThread(); 1021 PostSetNeedsCommitToMainThread();
1016 break; 1022 break;
1017 case 1: 1023 case 1:
1018 EXPECT_EQ(node->opacity, 0.75f); 1024 EXPECT_EQ(node->opacity, 0.75f);
1025 PostSetNeedsCommitToMainThread();
1026 break;
1027 case 2:
1028 EXPECT_EQ(node->opacity, 0.75f);
1019 impl->sync_tree()->root_layer_for_testing()->OnOpacityAnimated(0.75f); 1029 impl->sync_tree()->root_layer_for_testing()->OnOpacityAnimated(0.75f);
1020 PostSetNeedsCommitToMainThread(); 1030 PostSetNeedsCommitToMainThread();
1021 break; 1031 break;
1022 case 2: 1032 case 3:
1023 EXPECT_EQ(node->opacity, 0.5f); 1033 EXPECT_EQ(node->opacity, 0.5f);
1034 PostSetNeedsCommitToMainThread();
1035 break;
1036 case 4:
1037 EXPECT_EQ(node->opacity, 0.25f);
1024 EndTest(); 1038 EndTest();
1025 break; 1039 break;
1026 } 1040 }
1027 } 1041 }
1028 1042
1029 void AfterTest() override {} 1043 void AfterTest() override {}
1030 1044
1031 private: 1045 private:
1032 scoped_refptr<Layer> root_; 1046 scoped_refptr<Layer> root_;
1033 }; 1047 };
(...skipping 20 matching lines...) Expand all
1054 case 1: 1068 case 1:
1055 node->local = rotate10; 1069 node->local = rotate10;
1056 node->is_currently_animating = true; 1070 node->is_currently_animating = true;
1057 break; 1071 break;
1058 case 2: 1072 case 2:
1059 node->is_currently_animating = true; 1073 node->is_currently_animating = true;
1060 break; 1074 break;
1061 case 3: 1075 case 3:
1062 node->is_currently_animating = false; 1076 node->is_currently_animating = false;
1063 break; 1077 break;
1078 case 4:
1079 node->local = gfx::Transform();
1080 node->is_currently_animating = true;
1081 break;
1064 } 1082 }
1065 } 1083 }
1066 1084
1067 void CommitCompleteOnThread(LayerTreeHostImpl* impl) override { 1085 void CommitCompleteOnThread(LayerTreeHostImpl* impl) override {
1068 TransformTree& transform_tree = 1086 TransformTree& transform_tree =
1069 impl->sync_tree()->property_trees()->transform_tree; 1087 impl->sync_tree()->property_trees()->transform_tree;
1070 TransformNode* node = transform_tree.Node( 1088 TransformNode* node = transform_tree.Node(
1071 impl->sync_tree()->root_layer_for_testing()->transform_tree_index()); 1089 impl->sync_tree()->root_layer_for_testing()->transform_tree_index());
1072 gfx::Transform rotate10; 1090 gfx::Transform rotate10;
1073 rotate10.Rotate(10.f); 1091 rotate10.Rotate(10.f);
1074 gfx::Transform rotate20; 1092 gfx::Transform rotate20;
1075 rotate20.Rotate(20.f); 1093 rotate20.Rotate(20.f);
1076 switch (impl->sync_tree()->source_frame_number()) { 1094 switch (impl->sync_tree()->source_frame_number()) {
1077 case 0: 1095 case 0:
1078 impl->sync_tree()->root_layer_for_testing()->OnTransformAnimated( 1096 impl->sync_tree()->root_layer_for_testing()->OnTransformAnimated(
1079 rotate20); 1097 rotate20);
1080 PostSetNeedsCommitToMainThread(); 1098 PostSetNeedsCommitToMainThread();
1081 break; 1099 break;
1082 case 1: 1100 case 1:
1083 EXPECT_EQ(node->local, rotate20); 1101 EXPECT_EQ(node->local, rotate20);
1084 impl->sync_tree()->root_layer_for_testing()->OnTransformAnimated(
1085 rotate20);
1086 PostSetNeedsCommitToMainThread(); 1102 PostSetNeedsCommitToMainThread();
1087 break; 1103 break;
1088 case 2: 1104 case 2:
1089 EXPECT_EQ(node->local, rotate20); 1105 EXPECT_EQ(node->local, rotate20);
1090 impl->sync_tree()->root_layer_for_testing()->OnTransformAnimated( 1106 impl->sync_tree()->root_layer_for_testing()->OnTransformAnimated(
1091 rotate20); 1107 rotate20);
1092 PostSetNeedsCommitToMainThread(); 1108 PostSetNeedsCommitToMainThread();
1093 break; 1109 break;
1094 case 3: 1110 case 3:
1095 EXPECT_EQ(node->local, rotate10); 1111 EXPECT_EQ(node->local, rotate10);
1112 PostSetNeedsCommitToMainThread();
1113 break;
1114 case 4:
1115 EXPECT_EQ(node->local, gfx::Transform());
1096 EndTest(); 1116 EndTest();
1097 } 1117 }
1098 } 1118 }
1099 1119
1100 void AfterTest() override {} 1120 void AfterTest() override {}
1101 1121
1102 private: 1122 private:
1103 scoped_refptr<Layer> root_; 1123 scoped_refptr<Layer> root_;
1104 }; 1124 };
1105 1125
(...skipping 5767 matching lines...) Expand 10 before | Expand all | Expand 10 after
6873 private: 6893 private:
6874 FakeContentLayerClient client_; 6894 FakeContentLayerClient client_;
6875 const gfx::Size viewport_size_; 6895 const gfx::Size viewport_size_;
6876 const gfx::Size large_image_size_; 6896 const gfx::Size large_image_size_;
6877 }; 6897 };
6878 6898
6879 SINGLE_AND_MULTI_THREAD_TEST_F(GpuRasterizationSucceedsWithLargeImage); 6899 SINGLE_AND_MULTI_THREAD_TEST_F(GpuRasterizationSucceedsWithLargeImage);
6880 6900
6881 } // namespace 6901 } // namespace
6882 } // namespace cc 6902 } // namespace cc
OLDNEW
« no previous file with comments | « cc/layers/layer_impl.cc ('k') | cc/trees/layer_tree_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698