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

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

Issue 2877033002: Fix cc scrollbar layer issues with initialization, and use element ids throughout. (Closed)
Patch Set: none Created 3 years, 7 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/trees/property_tree.cc ('k') | third_party/WebKit/Source/core/animation/Animation.cpp » ('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 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 690 matching lines...) Expand 10 before | Expand all | Expand 10 after
701 } 701 }
702 702
703 static inline bool HideLayerAndSubtree(Layer* layer) { 703 static inline bool HideLayerAndSubtree(Layer* layer) {
704 return layer->hide_layer_and_subtree(); 704 return layer->hide_layer_and_subtree();
705 } 705 }
706 706
707 static inline bool HideLayerAndSubtree(LayerImpl* layer) { 707 static inline bool HideLayerAndSubtree(LayerImpl* layer) {
708 return layer->test_properties()->hide_layer_and_subtree; 708 return layer->test_properties()->hide_layer_and_subtree;
709 } 709 }
710 710
711 static inline bool AlwaysUseActiveTreeOpacity(Layer* layer) {
712 return layer->AlwaysUseActiveTreeOpacity();
713 }
714
715 static inline bool AlwaysUseActiveTreeOpacity(LayerImpl* layer) {
716 return false;
717 }
718
719 static inline bool HasCopyRequest(Layer* layer) { 711 static inline bool HasCopyRequest(Layer* layer) {
720 return layer->HasCopyRequest(); 712 return layer->HasCopyRequest();
721 } 713 }
722 714
723 static inline bool HasCopyRequest(LayerImpl* layer) { 715 static inline bool HasCopyRequest(LayerImpl* layer) {
724 return !layer->test_properties()->copy_requests.empty(); 716 return !layer->test_properties()->copy_requests.empty();
725 } 717 }
726 718
727 static inline bool PropertyChanged(Layer* layer) { 719 static inline bool PropertyChanged(Layer* layer) {
728 return layer->subtree_property_changed(); 720 return layer->subtree_property_changed();
(...skipping 182 matching lines...) Expand 10 before | Expand all | Expand 10 after
911 data_for_children->compound_transform_since_render_target *= 903 data_for_children->compound_transform_since_render_target *=
912 Transform(layer); 904 Transform(layer);
913 return false; 905 return false;
914 } 906 }
915 907
916 EffectTree& effect_tree = data_for_children->property_trees->effect_tree; 908 EffectTree& effect_tree = data_for_children->property_trees->effect_tree;
917 int node_id = effect_tree.Insert(EffectNode(), parent_id); 909 int node_id = effect_tree.Insert(EffectNode(), parent_id);
918 EffectNode* node = effect_tree.back(); 910 EffectNode* node = effect_tree.back();
919 911
920 node->owning_layer_id = layer->id(); 912 node->owning_layer_id = layer->id();
921 if (AlwaysUseActiveTreeOpacity(layer)) {
922 data_for_children->property_trees->always_use_active_tree_opacity_effect_ids
923 .push_back(node->owning_layer_id);
924 }
925
926 node->opacity = Opacity(layer); 913 node->opacity = Opacity(layer);
927 node->blend_mode = BlendMode(layer); 914 node->blend_mode = BlendMode(layer);
928 node->unscaled_mask_target_size = layer->bounds(); 915 node->unscaled_mask_target_size = layer->bounds();
929 node->has_render_surface = should_create_render_surface; 916 node->has_render_surface = should_create_render_surface;
930 node->has_copy_request = HasCopyRequest(layer); 917 node->has_copy_request = HasCopyRequest(layer);
931 node->filters = Filters(layer); 918 node->filters = Filters(layer);
932 node->background_filters = BackgroundFilters(layer); 919 node->background_filters = BackgroundFilters(layer);
933 node->filters_origin = FiltersOrigin(layer); 920 node->filters_origin = FiltersOrigin(layer);
934 node->has_potential_opacity_animation = has_potential_opacity_animation; 921 node->has_potential_opacity_animation = has_potential_opacity_animation;
935 node->has_potential_filter_animation = has_potential_filter_animation; 922 node->has_potential_filter_animation = has_potential_filter_animation;
(...skipping 456 matching lines...) Expand 10 before | Expand all | Expand 10 after
1392 root_layer, page_scale_layer, inner_viewport_scroll_layer, 1379 root_layer, page_scale_layer, inner_viewport_scroll_layer,
1393 outer_viewport_scroll_layer, overscroll_elasticity_layer, 1380 outer_viewport_scroll_layer, overscroll_elasticity_layer,
1394 elastic_overscroll, page_scale_factor, device_scale_factor, viewport, 1381 elastic_overscroll, page_scale_factor, device_scale_factor, viewport,
1395 device_transform, property_trees, color); 1382 device_transform, property_trees, color);
1396 property_trees->effect_tree.CreateOrReuseRenderSurfaces( 1383 property_trees->effect_tree.CreateOrReuseRenderSurfaces(
1397 &render_surfaces, root_layer->layer_tree_impl()); 1384 &render_surfaces, root_layer->layer_tree_impl());
1398 property_trees->ResetCachedData(); 1385 property_trees->ResetCachedData();
1399 } 1386 }
1400 1387
1401 } // namespace cc 1388 } // namespace cc
OLDNEW
« no previous file with comments | « cc/trees/property_tree.cc ('k') | third_party/WebKit/Source/core/animation/Animation.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698