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

Unified Diff: cc/trees/property_tree.h

Issue 2758343002: cc: Use Element Id to Record Animation Changes (Closed)
Patch Set: fix for test 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
« no previous file with comments | « cc/trees/layer_tree_impl.cc ('k') | cc/trees/property_tree.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/property_tree.h
diff --git a/cc/trees/property_tree.h b/cc/trees/property_tree.h
index 3dc0594bf158e558b5b37a49e9abadc9655dd58c..46f0cb7c57e68e9902183589a0bf55b20a912293 100644
--- a/cc/trees/property_tree.h
+++ b/cc/trees/property_tree.h
@@ -104,7 +104,8 @@ class CC_EXPORT PropertyTree {
}
T* UpdateNodeFromOwningLayerId(int id) {
int index = FindNodeIndexFromOwningLayerId(id);
- if (index == kInvalidNodeId && property_trees()->is_main_thread) {
+ if (index == kInvalidNodeId) {
+ DCHECK(property_trees()->is_main_thread);
property_trees()->needs_rebuild = true;
}
@@ -171,6 +172,7 @@ class CC_EXPORT TransformTree final : public PropertyTree<TransformNode> {
void clear();
+ TransformNode* FindNodeFromElementId(ElementId id);
void OnTransformAnimated(const gfx::Transform& transform,
int id,
LayerTreeImpl* layer_tree_impl);
@@ -340,6 +342,7 @@ class CC_EXPORT EffectTree final : public PropertyTree<EffectNode> {
void UpdateSurfaceContentsScale(EffectNode* node);
+ EffectNode* FindNodeFromElementId(ElementId id);
void OnOpacityAnimated(float opacity, int id, LayerTreeImpl* layer_tree_impl);
void OnFilterAnimated(const FilterOperations& filters,
int id,
« no previous file with comments | « cc/trees/layer_tree_impl.cc ('k') | cc/trees/property_tree.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698