From 22a55369c16d825d2f06df9be5bfeb44840ccab2 Mon Sep 17 00:00:00 2001 From: danakj Date: Tue, 10 Mar 2015 18:33:09 -0700 Subject: [PATCH] surfaces: Remove draw_and_swap_full_viewport_every_frame flag. The draw_and_swap_full_viewport_every_frame flag causes the compositor to try and draw even when its frame is incomplete. This was done in order to support forced resource reclaiming, because when forced resource reclaiming is on, we need to draw/swap even when there is no damage (since the surface host has released the resources). However, LayerTreeHostImpl::CalculateRenderPasses already checks can_force_reclaim_resources and will not early out for no damage when it is true. So this draw_and_swap_full_viewport_every_frame is not helping the resource reclaiming case, and causes is to draw incomplete frames instead. R=jbauman BUG=465861 Review URL: https://codereview.chromium.org/994223002 Cr-Commit-Position: refs/heads/master@{#320021} --- cc/surfaces/surface_display_output_surface.cc | 3 --- 1 file changed, 3 deletions(-) diff --git a/cc/surfaces/surface_display_output_surface.cc b/cc/surfaces/surface_display_output_surface.cc index 88e166c22496..ac60271aee5b 100644 --- a/cc/surfaces/surface_display_output_surface.cc +++ b/cc/surfaces/surface_display_output_surface.cc @@ -25,9 +25,6 @@ SurfaceDisplayOutputSurface::SurfaceDisplayOutputSurface( capabilities_.delegated_rendering = true; capabilities_.max_frames_pending = 1; capabilities_.can_force_reclaim_resources = true; - // Frame always needs to be swapped because forced resource reclaiming - // destroys the Display's copy. - capabilities_.draw_and_swap_full_viewport_every_frame = true; } SurfaceDisplayOutputSurface::~SurfaceDisplayOutputSurface() { -- 2.11.4.GIT