| Index: cc/trees/debug_rect_history.cc
|
| diff --git a/cc/trees/debug_rect_history.cc b/cc/trees/debug_rect_history.cc
|
| index 243c610214ca1ebf654af6c33f94bee7209d71b5..c07fca2c7458967c97c52867e4d9b1bc5e893ae2 100644
|
| --- a/cc/trees/debug_rect_history.cc
|
| +++ b/cc/trees/debug_rect_history.cc
|
| @@ -32,7 +32,7 @@ DebugRectHistory::~DebugRectHistory() {}
|
| void DebugRectHistory::SaveDebugRectsForCurrentFrame(
|
| LayerTreeImpl* tree_impl,
|
| LayerImpl* hud_layer,
|
| - const LayerImplList& render_surface_layer_list,
|
| + const RenderSurfaceList& render_surface_list,
|
| const LayerTreeDebugState& debug_state) {
|
| // For now, clear all rects from previous frames. In the future we may want to
|
| // store all debug rects for a history of many frames.
|
| @@ -54,13 +54,13 @@ void DebugRectHistory::SaveDebugRectsForCurrentFrame(
|
| SavePaintRects(tree_impl);
|
|
|
| if (debug_state.show_property_changed_rects)
|
| - SavePropertyChangedRects(render_surface_layer_list, hud_layer);
|
| + SavePropertyChangedRects(tree_impl, hud_layer);
|
|
|
| if (debug_state.show_surface_damage_rects)
|
| - SaveSurfaceDamageRects(render_surface_layer_list);
|
| + SaveSurfaceDamageRects(render_surface_list);
|
|
|
| if (debug_state.show_screen_space_rects)
|
| - SaveScreenSpaceRects(render_surface_layer_list);
|
| + SaveScreenSpaceRects(render_surface_list);
|
|
|
| if (debug_state.show_layer_animation_bounds_rects)
|
| SaveLayerAnimationBoundsRects(tree_impl);
|
| @@ -84,46 +84,27 @@ void DebugRectHistory::SavePaintRects(LayerTreeImpl* tree_impl) {
|
| }
|
| }
|
|
|
| -void DebugRectHistory::SavePropertyChangedRects(
|
| - const LayerImplList& render_surface_layer_list,
|
| - LayerImpl* hud_layer) {
|
| - for (size_t i = 0; i < render_surface_layer_list.size(); ++i) {
|
| - size_t surface_index = render_surface_layer_list.size() - 1 - i;
|
| - LayerImpl* render_surface_layer = render_surface_layer_list[surface_index];
|
| - RenderSurfaceImpl* render_surface =
|
| - render_surface_layer->GetRenderSurface();
|
| - DCHECK(render_surface);
|
| -
|
| - const LayerImplList& layer_list = render_surface->layer_list();
|
| - for (unsigned layer_index = 0; layer_index < layer_list.size();
|
| - ++layer_index) {
|
| - LayerImpl* layer = layer_list[layer_index];
|
| -
|
| - if (layer->GetRenderSurface() &&
|
| - layer->GetRenderSurface() != render_surface)
|
| - continue;
|
| -
|
| - if (layer == hud_layer)
|
| - continue;
|
| +void DebugRectHistory::SavePropertyChangedRects(LayerTreeImpl* tree_impl,
|
| + LayerImpl* hud_layer) {
|
| + for (LayerImpl* layer : *tree_impl) {
|
| + if (layer == hud_layer)
|
| + continue;
|
|
|
| - if (!layer->LayerPropertyChanged())
|
| - continue;
|
| + if (!layer->LayerPropertyChanged())
|
| + continue;
|
|
|
| - debug_rects_.push_back(DebugRect(
|
| - PROPERTY_CHANGED_RECT_TYPE,
|
| - MathUtil::MapEnclosingClippedRect(layer->ScreenSpaceTransform(),
|
| - gfx::Rect(layer->bounds()))));
|
| - }
|
| + debug_rects_.push_back(DebugRect(
|
| + PROPERTY_CHANGED_RECT_TYPE,
|
| + MathUtil::MapEnclosingClippedRect(layer->ScreenSpaceTransform(),
|
| + gfx::Rect(layer->bounds()))));
|
| }
|
| }
|
|
|
| void DebugRectHistory::SaveSurfaceDamageRects(
|
| - const LayerImplList& render_surface_layer_list) {
|
| - for (size_t i = 0; i < render_surface_layer_list.size(); ++i) {
|
| - size_t surface_index = render_surface_layer_list.size() - 1 - i;
|
| - LayerImpl* render_surface_layer = render_surface_layer_list[surface_index];
|
| - RenderSurfaceImpl* render_surface =
|
| - render_surface_layer->GetRenderSurface();
|
| + const RenderSurfaceList& render_surface_list) {
|
| + for (size_t i = 0; i < render_surface_list.size(); ++i) {
|
| + size_t surface_index = render_surface_list.size() - 1 - i;
|
| + RenderSurfaceImpl* render_surface = render_surface_list[surface_index];
|
| DCHECK(render_surface);
|
|
|
| debug_rects_.push_back(DebugRect(
|
| @@ -134,12 +115,10 @@ void DebugRectHistory::SaveSurfaceDamageRects(
|
| }
|
|
|
| void DebugRectHistory::SaveScreenSpaceRects(
|
| - const LayerImplList& render_surface_layer_list) {
|
| - for (size_t i = 0; i < render_surface_layer_list.size(); ++i) {
|
| - size_t surface_index = render_surface_layer_list.size() - 1 - i;
|
| - LayerImpl* render_surface_layer = render_surface_layer_list[surface_index];
|
| - RenderSurfaceImpl* render_surface =
|
| - render_surface_layer->GetRenderSurface();
|
| + const RenderSurfaceList& render_surface_list) {
|
| + for (size_t i = 0; i < render_surface_list.size(); ++i) {
|
| + size_t surface_index = render_surface_list.size() - 1 - i;
|
| + RenderSurfaceImpl* render_surface = render_surface_list[surface_index];
|
| DCHECK(render_surface);
|
|
|
| debug_rects_.push_back(DebugRect(
|
|
|