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

Issue 771357 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
inactive
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature

Blocking:
issue 730193
issue 775030



Sign in to add a comment

Viz: Consider LatencyInfo plumbing in RWHImpl::SubmitCompositorFrame

Project Member Reported by fsamuel@google.com, Oct 3 2017

Issue description

RenderWidgetHostLatencyTracker latency_tracker_ lives in RenderWidgetHostImpl.

When we receive a CompositorFrame in RenderWidgetHostImpl::SubmitCompositorFrame, we add to its LatencyInfo.

latency_tracker_.OnSwapCompositorFrame(&frame.metadata.latency_info);

This cannot work with Viz. 

Maybe we can add a privileged host interface to Viz (A LatencyInfoObserver) that observes latency infos in frames, and passes them along to the host?

This seems expensive. Maybe there's a better way to propagate this? Perhaps once per aggregation?
 

Comment 1 by fsamuel@google.com, Oct 3 2017

Components: Internals>Compositing

Comment 2 by ericrk@chromium.org, Oct 11 2017

Labels: -Type-Bug Type-Feature
Status: Available (was: Untriaged)

Comment 3 by fsamuel@google.com, Oct 16 2017

Owner: mfomitchev@chromium.org
Status: Assigned (was: Available)

Comment 4 by fsamuel@google.com, Oct 16 2017

Blocking: 775030
Status: Fixed (was: Assigned)
This is plumbed in Viz here: https://cs.chromium.org/chromium/src/components/viz/service/frame_sinks/compositor_frame_sink_support.cc?sq=package:chromium&dr=CSs&l=176
LatencyTracker is owned by DisplayOutputSUrface, and it takes care of reporting a bunch of UMA and UKM metrics.

So closing this as fixed.

Sign in to add a comment