diff options
author | piman@chromium.org <piman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-08 16:02:38 +0000 |
---|---|---|
committer | piman@chromium.org <piman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-08 16:02:38 +0000 |
commit | 2ae13edf5976610500da81d2b29ba5315dba5ff1 (patch) | |
tree | b38b82917a0f116cec9670709681f39bda40903f /cc/trees/thread_proxy.h | |
parent | 9dffa619d6e1dd3647e811eb984bb89ee2ef8754 (diff) | |
download | chromium_src-2ae13edf5976610500da81d2b29ba5315dba5ff1.zip chromium_src-2ae13edf5976610500da81d2b29ba5315dba5ff1.tar.gz chromium_src-2ae13edf5976610500da81d2b29ba5315dba5ff1.tar.bz2 |
DelegatingRenderer: send frame on SwapBuffers instead of DrawFrame
CompositeAndReadback, though not implemented, should stay consistent. If we send
the frame on DrawFrame, we will get an Ack from the browser, even though the
frame rate controller doesn't expect one in that case. If that happens,
num_frames_pending_ becomes negative and the frame rate throttling doesn't work
any more, causing bad frames.
Instead, be consistent with GLRenderer and send the frame on SwapBuffers.
BUG=123444
TEST=LayerTreeHostTestNumFramesPending
Review URL: https://chromiumcodereview.appspot.com/14941005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198913 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/trees/thread_proxy.h')
-rw-r--r-- | cc/trees/thread_proxy.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/cc/trees/thread_proxy.h b/cc/trees/thread_proxy.h index 21ea708..981157f 100644 --- a/cc/trees/thread_proxy.h +++ b/cc/trees/thread_proxy.h @@ -111,6 +111,10 @@ class ThreadProxy : public Proxy, return scheduler_on_impl_thread_->MaxFramesPending(); } + int NumFramesPendingForTesting() const { + return scheduler_on_impl_thread_->NumFramesPendingForTesting(); + } + private: ThreadProxy(LayerTreeHost* layer_tree_host, scoped_ptr<Thread> impl_thread); |