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

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

Issue 2690753002: cc: Move render surface ownership from layers to the effect tree (Closed)
Patch Set: Only update surfaces when can_render_to_separate_surface changes Created 3 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
« no previous file with comments | « cc/trees/effect_node.h ('k') | cc/trees/layer_tree_host_common.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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "base/trace_event/trace_event_argument.h" 5 #include "base/trace_event/trace_event_argument.h"
6 #include "cc/layers/layer.h" 6 #include "cc/layers/layer.h"
7 #include "cc/trees/effect_node.h" 7 #include "cc/trees/effect_node.h"
8 #include "cc/trees/property_tree.h" 8 #include "cc/trees/property_tree.h"
9 9
10 namespace cc { 10 namespace cc {
11 11
12 EffectNode::EffectNode() 12 EffectNode::EffectNode()
13 : id(EffectTree::kInvalidNodeId), 13 : id(EffectTree::kInvalidNodeId),
14 parent_id(EffectTree::kInvalidNodeId), 14 parent_id(EffectTree::kInvalidNodeId),
15 owning_layer_id(Layer::INVALID_ID), 15 owning_layer_id(Layer::INVALID_ID),
16 opacity(1.f), 16 opacity(1.f),
17 screen_space_opacity(1.f), 17 screen_space_opacity(1.f),
18 blend_mode(SkBlendMode::kSrcOver), 18 blend_mode(SkBlendMode::kSrcOver),
19 has_render_surface(false), 19 has_render_surface(false),
20 render_surface(nullptr),
21 surface_is_clipped(false), 20 surface_is_clipped(false),
22 has_copy_request(false), 21 has_copy_request(false),
23 hidden_by_backface_visibility(false), 22 hidden_by_backface_visibility(false),
24 double_sided(false), 23 double_sided(false),
25 is_drawn(true), 24 is_drawn(true),
26 subtree_hidden(false), 25 subtree_hidden(false),
27 has_potential_filter_animation(false), 26 has_potential_filter_animation(false),
28 has_potential_opacity_animation(false), 27 has_potential_opacity_animation(false),
29 is_currently_animating_filter(false), 28 is_currently_animating_filter(false),
30 is_currently_animating_opacity(false), 29 is_currently_animating_opacity(false),
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 value->SetBoolean("effect_changed", effect_changed); 83 value->SetBoolean("effect_changed", effect_changed);
85 value->SetInteger("num_copy_requests_in_subtree", 84 value->SetInteger("num_copy_requests_in_subtree",
86 num_copy_requests_in_subtree); 85 num_copy_requests_in_subtree);
87 value->SetInteger("transform_id", transform_id); 86 value->SetInteger("transform_id", transform_id);
88 value->SetInteger("clip_id", clip_id); 87 value->SetInteger("clip_id", clip_id);
89 value->SetInteger("target_id", target_id); 88 value->SetInteger("target_id", target_id);
90 value->SetInteger("mask_layer_id", mask_layer_id); 89 value->SetInteger("mask_layer_id", mask_layer_id);
91 } 90 }
92 91
93 } // namespace cc 92 } // namespace cc
OLDNEW
« no previous file with comments | « cc/trees/effect_node.h ('k') | cc/trees/layer_tree_host_common.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698