New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 912730 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Dec 6
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----



Sign in to add a comment

Add a way to link raster tasks to their corresponding frames

Project Member Reported by lwsong@google.com, Dec 6

Issue description

In order to measure the time difference between when raster tasks are run and when compositor frames are submitted, there needs to be a way to link raster tasks to their corresponding compositor frames. 
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 11

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

commit b7f402a95386e6e0c9e31850b0d5d356d5258f8b
Author: Linwan Song <lwsong@google.com>
Date: Tue Dec 11 17:50:57 2018

Link raster tasks to frames by adding frame ids as a tracing argument.

This change links tile raster tasks to the tile manager that
initiated them. Tile managers are in turn linked to a frame,
which is processed by telemetry later on to determine the time
difference between when raster tasks start to when the
compositor frame is submitted.

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
Change-Id: I876a029f6db1e664aafe945fcdd629da3bdc681e
Bug:  912730 
Reviewed-on: https://chromium-review.googlesource.com/c/1286889
Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: weiliangc <weiliangc@chromium.org>
Reviewed-by: Yi Xu <yiyix@chromium.org>
Cr-Commit-Position: refs/heads/master@{#615572}
[modify] https://crrev.com/b7f402a95386e6e0c9e31850b0d5d356d5258f8b/cc/mojo_embedder/async_layer_tree_frame_sink.cc
[modify] https://crrev.com/b7f402a95386e6e0c9e31850b0d5d356d5258f8b/cc/raster/task.h
[modify] https://crrev.com/b7f402a95386e6e0c9e31850b0d5d356d5258f8b/cc/tiles/tile_manager.cc
[modify] https://crrev.com/b7f402a95386e6e0c9e31850b0d5d356d5258f8b/cc/trees/layer_tree_frame_sink.h
[modify] https://crrev.com/b7f402a95386e6e0c9e31850b0d5d356d5258f8b/cc/trees/layer_tree_host_impl.cc
[modify] https://crrev.com/b7f402a95386e6e0c9e31850b0d5d356d5258f8b/content/renderer/categorized_worker_pool.cc

Sign in to add a comment