Commit 1e6ffca4 authored by Eric Karl's avatar Eric Karl Committed by Commit Bot

Disable latency recovery for Android Surface Sync

CC's latency recovery can lead to signficant jank in cases where
latency can't effectively be recovered. This seems to happen in both
OOP-D and non-OOP-D, but to minimize risk, disabling in OOP-D only for
now. We should re-evaluate this feature and potentially rework or
remove it.

TBRing piman so that this change can begin baking over the weekend for
potential merge. Discussed this approach with sunnyps previously.

TBR=sunnyps
TBR=piman

Bug: 933846
Change-Id: Icbce2ef784d03adb3ce233e816b5fd1361191de4
Reviewed-on: https://chromium-review.googlesource.com/c/1485231
Commit-Queue: Eric Karl <ericrk@chromium.org>
Reviewed-by: default avatarEric Karl <ericrk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#634981}
parent 8e5286cc
...@@ -3074,6 +3074,14 @@ cc::LayerTreeSettings RenderWidget::GenerateLayerTreeSettings( ...@@ -3074,6 +3074,14 @@ cc::LayerTreeSettings RenderWidget::GenerateLayerTreeSettings(
settings.wait_for_all_pipeline_stages_before_draw = true; settings.wait_for_all_pipeline_stages_before_draw = true;
settings.enable_latency_recovery = false; settings.enable_latency_recovery = false;
} }
#if defined(OS_ANDROID)
if (features::IsSurfaceSynchronizationEnabled()) {
// TODO(crbug.com/933846): LatencyRecovery is causing jank on Android.
// Disable in viz mode for now, with plant o disable more widely once
// viz launches.
settings.enable_latency_recovery = false;
}
#endif
settings.enable_image_animation_resync = settings.enable_image_animation_resync =
!cmd.HasSwitch(switches::kDisableImageAnimationResync); !cmd.HasSwitch(switches::kDisableImageAnimationResync);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment