New issue
Advanced search Search tips

Issue 825912 link

Starred by 1 user
Status: Fixed
Owner:
pdr@chromium.org
Closed: Mar 2018
Cc:
wangxianzhu@chromium.org
vmp...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Create a unified Compositing.Renderer.LayersUpdateTime metric

Project Member Reported by pdr@chromium.org, Mar 26 2018 
Compositing.Renderer.LayersUpdateTime.* is split into 5 buckets which isn't useful when tracking the overall performance of LayersUpdateTime. For example, SPV175 is putting PaintChunksToCcLayer in there and we'd like to know the overall effect.
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 27 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/80c4ae5c1d5b1ff6c51e8dad57cff3418ce8184d

commit 80c4ae5c1d5b1ff6c51e8dad57cff3418ce8184d
Author: Philip Rogers <pdr@chromium.org>
Date: Tue Mar 27 19:40:26 2018

[CI] Track total Compositing.{Browser,Renderer}.LayersUpdateTime

We current have a histogram for LayersUpdateTime but it is bucketed
by the layer count. This patch adds a new metric for the total
LayersUpdateTime that is not bucketed.

Bug:  825912 
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: Ie9d83f90abf529af97bc2393f8f3ad6dc2e3530d
Reviewed-on: https://chromium-review.googlesource.com/981262
Reviewed-by: Majid Valipour <majidvp@chromium.org>
Reviewed-by: vmpstr <vmpstr@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Commit-Queue: vmpstr <vmpstr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#546208}
[modify] https://crrev.com/80c4ae5c1d5b1ff6c51e8dad57cff3418ce8184d/cc/trees/layer_tree_host.cc
[modify] https://crrev.com/80c4ae5c1d5b1ff6c51e8dad57cff3418ce8184d/tools/metrics/histograms/histograms.xml

Comment 2 by pdr@chromium.org, Mar 27 2018

Status: Fixed (was: Assigned)
►

Sign in to add a comment