diff options
author | brianderson <brianderson@chromium.org> | 2015-11-17 19:41:15 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-11-18 03:41:57 +0000 |
commit | 0055b7eaee4ca249cb83ac8ddc5200cbf7a12069 (patch) | |
tree | 28c8175c71eb6c6831bc2e7903679233db2ec8bd /blimp | |
parent | 27b48cde7e6243c777ff7b44c9fe7d341c0af2d2 (diff) | |
download | chromium_src-0055b7eaee4ca249cb83ac8ddc5200cbf7a12069.zip chromium_src-0055b7eaee4ca249cb83ac8ddc5200cbf7a12069.tar.gz chromium_src-0055b7eaee4ca249cb83ac8ddc5200cbf7a12069.tar.bz2 |
cc: Clean up max frames/swaps pending usage.
All output surfaces used by the Renderer and UI should
only need a max swaps pending of 1, especially considering
Surfaces and CompositorTimingHistory don't support
multiple queued frames.
The output suface used by the cc::Display, however, may
have more than 1 swap pending - especially on platforms like
CrOS where the SwapAck is deferred until the buffer is actually
displayed.
This patch:
1) Changes the default max pending frames/swaps from 2 to 1.
2) Changes Blimp to have only 1 pending frame.
3) DCHECKS that all Renderers and UIs have a max swaps pending of 1.
4) Sets the default value in the constructor of
OutpuSurface::Capabilities, rather than through
an extra check for zero + init.
BUG=525756
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1435133004
Cr-Commit-Position: refs/heads/master@{#360283}
Diffstat (limited to 'blimp')
-rw-r--r-- | blimp/client/compositor/blimp_output_surface.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/blimp/client/compositor/blimp_output_surface.cc b/blimp/client/compositor/blimp_output_surface.cc index dd4b89a..7c2836e 100644 --- a/blimp/client/compositor/blimp_output_surface.cc +++ b/blimp/client/compositor/blimp_output_surface.cc @@ -14,7 +14,6 @@ namespace blimp { BlimpOutputSurface::BlimpOutputSurface( const scoped_refptr<cc::ContextProvider>& context_provider) : cc::OutputSurface(context_provider) { - capabilities_.max_frames_pending = 2; } BlimpOutputSurface::~BlimpOutputSurface() {} |