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

Unified Diff: cc/trees/layer_tree_host_common.cc

Issue 1437413002: cc: Remove ScopedPtrVector and cc::remove_if. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 1 month 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_host_common.h ('k') | cc/trees/layer_tree_host_common_perftest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/layer_tree_host_common.cc
diff --git a/cc/trees/layer_tree_host_common.cc b/cc/trees/layer_tree_host_common.cc
index dddb80c5ff90d87e3d1fb0f90daafc1477e2b256..c9c74954c08f50a38364535db8c9b055745cb9a7 100644
--- a/cc/trees/layer_tree_host_common.cc
+++ b/cc/trees/layer_tree_host_common.cc
@@ -2574,23 +2574,23 @@ void CalculateRenderSurfaceLayerListInternal(
if (compute_content_rects && render_to_separate_surface)
layer->render_surface()->SetAccumulatedContentRect(gfx::Rect());
- for (auto& child_layer : layer->children()) {
+ for (const auto& child_layer : layer->children()) {
CalculateRenderSurfaceLayerListInternal(
- child_layer, property_trees, render_surface_layer_list, descendants,
- nearest_occlusion_immune_ancestor, layer_is_drawn,
+ child_layer.get(), property_trees, render_surface_layer_list,
+ descendants, nearest_occlusion_immune_ancestor, layer_is_drawn,
can_render_to_separate_surface, current_render_surface_layer_list_id,
max_texture_size, verify_property_trees, use_property_trees);
// If the child is its own render target, then it has a render surface.
- if (child_layer->render_target() == child_layer &&
+ if (child_layer->render_target() == child_layer.get() &&
!child_layer->render_surface()->layer_list().empty() &&
!child_layer->render_surface()->content_rect().IsEmpty()) {
// This child will contribute its render surface, which means
// we need to mark just the mask layer (and replica mask layer)
// with the id.
MarkMasksWithRenderSurfaceLayerListId(
- child_layer, current_render_surface_layer_list_id);
- descendants->push_back(child_layer);
+ child_layer.get(), current_render_surface_layer_list_id);
+ descendants->push_back(child_layer.get());
}
if (child_layer->layer_or_descendant_is_drawn()) {
« no previous file with comments | « cc/trees/layer_tree_host_common.h ('k') | cc/trees/layer_tree_host_common_perftest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698