New issue
Advanced search Search tips

Issue 725344 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Feature

Blocking:
issue 725351



Sign in to add a comment

Add UMA for tracking decode times for out of raster and raster tasks.

Project Member Reported by khushals...@chromium.org, May 23 2017

Issue description

With checker-imaging, we expect to see a decline in image decode tasks that block raster. Currently the metrics that track the length of image decodes are Renderer4.ImageDecodeTaskDurationUs.Software/Gpu. We should update them to separately track decodes for out of raster tasks and bucket the clients of these tasks (decode API versus checker-imaging) separately.
 
Blocking: 725351

Comment 2 by vmp...@chromium.org, May 23 2017

Labels: -Type-Bug Type-Feature
Owner: khushals...@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 2 2017

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

commit 97474434c7f06805f9bdc97c2848ef25cf07f550
Author: khushalsagar <khushalsagar@chromium.org>
Date: Fri Jun 02 02:35:33 2017

cc: Add UMA for tracking decode duration for out of raster decodes.

We currently bucket UMA for all decode tasks in a single histogram.
With checker-imaging enabled, we expect to see a decline in the decode
duration for images decoded with raster. Add UMA to track these
seperately.

BUG= 725344 
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2904743002
Cr-Commit-Position: refs/heads/master@{#476527}

[modify] https://crrev.com/97474434c7f06805f9bdc97c2848ef25cf07f550/cc/base/devtools_instrumentation.cc
[modify] https://crrev.com/97474434c7f06805f9bdc97c2848ef25cf07f550/cc/base/devtools_instrumentation.h
[modify] https://crrev.com/97474434c7f06805f9bdc97c2848ef25cf07f550/cc/tiles/gpu_image_decode_cache.cc
[modify] https://crrev.com/97474434c7f06805f9bdc97c2848ef25cf07f550/cc/tiles/image_decode_cache.h
[modify] https://crrev.com/97474434c7f06805f9bdc97c2848ef25cf07f550/cc/tiles/software_image_decode_cache.cc
[modify] https://crrev.com/97474434c7f06805f9bdc97c2848ef25cf07f550/cc/tiles/tile_manager.cc
[modify] https://crrev.com/97474434c7f06805f9bdc97c2848ef25cf07f550/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Assigned)

Sign in to add a comment