cc: Adding DidFinishImplFrame to LTHI.
commit2caee4f89de17858822c11379da9c6cd80f90708
authormithro <mithro@mithis.com>
Thu, 7 May 2015 02:50:24 +0000 (6 19:50 -0700)
committerCommit bot <commit-bot@chromium.org>
Thu, 7 May 2015 02:51:49 +0000 (7 02:51 +0000)
tree775c42f5e389c195af036484c59f38e1563f7935
parentc463ab98438668e8e824f0ae3fa9a462b7724996
cc: Adding DidFinishImplFrame to LTHI.

This change moves the responsibility of clearing state inside the LTHI after an
impl frame is finished from the thread proxies to the LTHI class. This makes
WillBeginImplFrame and DidFinishImplFrame a logical pair.

This CL also;
 * Adds a test that checks the number of WillBeginImplFrame
   calls matches the DidFinishImplFrame calls.
 * Cleans up classes in cc/test/layer_tree_test.h around WillBeginImplFrame.
 * Removes the UpdateCurrentBeginFrameArgs / ResetCurrentBeginFrameArgs methods.
 * Fixes a bug where STP was calling ResetCurrentBeginFrameArgs twice every frame.

BUG=346230,481810
R=brianderson,enne

Review URL: https://codereview.chromium.org/1111743002

Cr-Commit-Position: refs/heads/master@{#328689}
14 files changed:
cc/resources/tile_manager_perftest.cc
cc/scheduler/scheduler.cc
cc/scheduler/scheduler.h
cc/scheduler/scheduler_unittest.cc
cc/test/layer_tree_test.cc
cc/test/layer_tree_test.h
cc/trees/layer_tree_host_impl.cc
cc/trees/layer_tree_host_impl.h
cc/trees/layer_tree_host_impl_unittest.cc
cc/trees/layer_tree_host_unittest.cc
cc/trees/single_thread_proxy.cc
cc/trees/single_thread_proxy.h
cc/trees/thread_proxy.cc
cc/trees/thread_proxy.h