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

Unified Diff: cc/layers/render_surface_impl.cc

Issue 2834123002: cc: Replace render surface layer list with a render surface list (Closed)
Patch Set: . Created 3 years, 8 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
Index: cc/layers/render_surface_impl.cc
diff --git a/cc/layers/render_surface_impl.cc b/cc/layers/render_surface_impl.cc
index fc05a4e6dee59052179b009ea85efe6d4a4cf1f5..82750774529327aa14f1670e91ee73055e121acd 100644
--- a/cc/layers/render_surface_impl.cc
+++ b/cc/layers/render_surface_impl.cc
@@ -38,10 +38,12 @@ RenderSurfaceImpl::RenderSurfaceImpl(LayerTreeImpl* layer_tree_impl,
: layer_tree_impl_(layer_tree_impl),
stable_effect_id_(stable_effect_id),
effect_tree_index_(EffectTree::kInvalidNodeId),
+ num_contributors_(0),
has_contributing_layer_that_escapes_clip_(false),
surface_property_changed_(false),
ancestor_property_changed_(false),
contributes_to_drawn_surface_(false),
+ is_render_surface_list_member_(false),
nearest_occlusion_immune_ancestor_(nullptr) {
damage_tracker_ = DamageTracker::Create();
}
@@ -352,16 +354,12 @@ void RenderSurfaceImpl::ResetPropertyChangedFlags() {
ancestor_property_changed_ = false;
}
-void RenderSurfaceImpl::ClearLayerLists() {
- layer_list_.clear();
-}
-
int RenderSurfaceImpl::GetRenderPassId() {
return id();
}
std::unique_ptr<RenderPass> RenderSurfaceImpl::CreateRenderPass() {
- std::unique_ptr<RenderPass> pass = RenderPass::Create(layer_list_.size());
+ std::unique_ptr<RenderPass> pass = RenderPass::Create(num_contributors_);
gfx::Rect damage_rect = GetDamageRect();
damage_rect.Intersect(content_rect());
pass->SetNew(id(), content_rect(), damage_rect,

Powered by Google App Engine
This is Rietveld 408576698