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

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

Issue 1698813002: CC Animation: Expose TargetProperty enum to be aliased in Blink Platform. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use custom hash in unordered_set. Created 4 years, 10 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/property_tree_builder.h" 5 #include "cc/trees/property_tree_builder.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <map> 9 #include <map>
10 #include <set> 10 #include <set>
(...skipping 223 matching lines...) Expand 10 before | Expand all | Expand 10 after
234 !layer->transform().IsIdentityOr2DTranslation(); 234 !layer->transform().IsIdentityOr2DTranslation();
235 235
236 const bool has_potentially_animated_transform = 236 const bool has_potentially_animated_transform =
237 layer->HasPotentiallyRunningTransformAnimation(); 237 layer->HasPotentiallyRunningTransformAnimation();
238 238
239 // A transform node is needed even for a finished animation, since differences 239 // A transform node is needed even for a finished animation, since differences
240 // in the timing of animation state updates can mean that an animation that's 240 // in the timing of animation state updates can mean that an animation that's
241 // in the Finished state at tree-building time on the main thread is still in 241 // in the Finished state at tree-building time on the main thread is still in
242 // the Running state right after commit on the compositor thread. 242 // the Running state right after commit on the compositor thread.
243 const bool has_any_transform_animation = 243 const bool has_any_transform_animation =
244 layer->HasAnyAnimationTargetingProperty(Animation::TRANSFORM); 244 layer->HasAnyAnimationTargetingProperty(
245 AnimationTargetProperty::TRANSFORM);
245 246
246 const bool has_surface = created_render_surface; 247 const bool has_surface = created_render_surface;
247 248
248 // A transform node is needed to change the render target for subtree when 249 // A transform node is needed to change the render target for subtree when
249 // a scroll child's render target is different from the scroll parent's render 250 // a scroll child's render target is different from the scroll parent's render
250 // target. 251 // target.
251 const bool scroll_child_has_different_target = 252 const bool scroll_child_has_different_target =
252 layer->scroll_parent() && 253 layer->scroll_parent() &&
253 layer->parent()->effect_tree_index() != 254 layer->parent()->effect_tree_index() !=
254 layer->scroll_parent()->effect_tree_index(); 255 layer->scroll_parent()->effect_tree_index();
(...skipping 697 matching lines...) Expand 10 before | Expand all | Expand 10 after
952 const gfx::Transform& device_transform, 953 const gfx::Transform& device_transform,
953 PropertyTrees* property_trees) { 954 PropertyTrees* property_trees) {
954 BuildPropertyTreesTopLevelInternal( 955 BuildPropertyTreesTopLevelInternal(
955 root_layer, page_scale_layer, inner_viewport_scroll_layer, 956 root_layer, page_scale_layer, inner_viewport_scroll_layer,
956 outer_viewport_scroll_layer, overscroll_elasticity_layer, 957 outer_viewport_scroll_layer, overscroll_elasticity_layer,
957 elastic_overscroll, page_scale_factor, device_scale_factor, viewport, 958 elastic_overscroll, page_scale_factor, device_scale_factor, viewport,
958 device_transform, property_trees); 959 device_transform, property_trees);
959 } 960 }
960 961
961 } // namespace cc 962 } // namespace cc
OLDNEW
« cc/animation/layer_animation_controller.h ('K') | « cc/trees/layer_tree_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698