Commit 2e6973e4 authored by ajuma's avatar ajuma Committed by Commit bot

cc: Fix badly-scoped trace event for CDP in LTH::DoUpdateLayers

The CalcDrawProps trace event in DoUpdateLayers actually measures
almost all of DoUpdateLayers. This CL moves it to the right scope
and renames it to BuildPropertyTrees.

BUG=581774
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review URL: https://codereview.chromium.org/1641023002

Cr-Commit-Position: refs/heads/master@{#372092}
parent eb448377
...@@ -820,17 +820,14 @@ bool LayerTreeHost::DoUpdateLayers(Layer* root_layer) { ...@@ -820,17 +820,14 @@ bool LayerTreeHost::DoUpdateLayers(Layer* root_layer) {
device_scale_factor_); device_scale_factor_);
} }
TRACE_EVENT0("cc", "LayerTreeHost::UpdateLayers::CalcDrawProps");
LayerTreeHostCommon::PreCalculateMetaInformation(root_layer);
gfx::Transform identity_transform; gfx::Transform identity_transform;
LayerList update_layer_list; LayerList update_layer_list;
{ {
TRACE_EVENT0("cc", "LayerTreeHost::UpdateLayers::BuildPropertyTrees");
TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("cc.debug.cdp-perf"), TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("cc.debug.cdp-perf"),
"LayerTreeHostCommon::ComputeVisibleRectsWithPropertyTrees"); "LayerTreeHostCommon::ComputeVisibleRectsWithPropertyTrees");
LayerTreeHostCommon::PreCalculateMetaInformation(root_layer);
bool can_render_to_separate_surface = true; bool can_render_to_separate_surface = true;
BuildPropertyTreesAndComputeVisibleRects( BuildPropertyTreesAndComputeVisibleRects(
root_layer, page_scale_layer, inner_viewport_scroll_layer_.get(), root_layer, page_scale_layer, inner_viewport_scroll_layer_.get(),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment