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

Issue 710517 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Surface synchronization: consider immediately activating late arriving CompositorFrames

Project Member Reported by fsam...@chromium.org, Apr 11 2017

Issue description

With surface synchronization, a set of clients are given a deadline to synchronize frames. If a client's CompositorFrame arrives late (after the deadline), then it'll be delayed until the next deadline (4 frames later). This causes a lot of latency and more gutter than necessary. We should probably track the set of unresolved dependencies on deadline activation, and immediately activate late arrivals. 
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 12 2017

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

commit 35927811f7b1d449675cbad2de6952844a93ef3e
Author: fsamuel <fsamuel@chromium.org>
Date: Wed Apr 12 19:59:49 2017

Surface synchronization: Immediately activate late arriving CompositorFrames

A dependent CompositorFrame may arrive late (after the deadline has hit and the
parent CompositorFrame activated). This CL tracks late arrivals (until the next
deadline hits) and immediately activates them despite dependencies.

This reduces latency and guttering of browser UI in Mus+Ash because this
prevents browser UI from blocking for too long on the renderer.

BUG= 672962 ,  710517 
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

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

[modify] https://crrev.com/35927811f7b1d449675cbad2de6952844a93ef3e/cc/surfaces/compositor_frame_sink_support_unittest.cc
[modify] https://crrev.com/35927811f7b1d449675cbad2de6952844a93ef3e/cc/surfaces/surface_dependency_tracker.cc
[modify] https://crrev.com/35927811f7b1d449675cbad2de6952844a93ef3e/cc/surfaces/surface_dependency_tracker.h

Owner: fsam...@chromium.org
Status: Fixed (was: Available)
Blocking: -601863
Components: -Internals>MUS Internals>Services>WindowService

Sign in to add a comment