Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4161)

Unified Diff: cc/test/layer_tree_test.cc

Issue 22926024: cc: Control activation from the Scheduler (Closed) Base URL: http://git.chromium.org/chromium/src.git@schedOutputSurface4
Patch Set: BlockNotifyReadyToActivateForTesting Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/test/layer_tree_test.cc
diff --git a/cc/test/layer_tree_test.cc b/cc/test/layer_tree_test.cc
index 6795f90dfb651d314b91c50d0bafebb7a1969cf9..2dfc1ba400476fbec2d51fca6c259cdf0f11d3b6 100644
--- a/cc/test/layer_tree_test.cc
+++ b/cc/test/layer_tree_test.cc
@@ -37,14 +37,6 @@ bool TestHooks::PrepareToDrawOnThread(LayerTreeHostImpl* host_impl,
return true;
}
-bool TestHooks::CanActivatePendingTree(LayerTreeHostImpl* host_impl) {
- return true;
-}
-
-bool TestHooks::CanActivatePendingTreeIfNeeded(LayerTreeHostImpl* host_impl) {
- return true;
-}
-
// Adapts LayerTreeHostImpl for test. Runs real code, then invokes test hooks.
class LayerTreeHostImplForTesting : public LayerTreeHostImpl {
public:
@@ -75,6 +67,11 @@ class LayerTreeHostImplForTesting : public LayerTreeHostImpl {
stats_instrumentation),
test_hooks_(test_hooks) {}
+ virtual void BeginFrame(const BeginFrameArgs& args) OVERRIDE {
+ test_hooks_->WillBeginFrameOnThread(this, args);
+ LayerTreeHostImpl::BeginFrame(args);
+ }
+
virtual void BeginCommit() OVERRIDE {
LayerTreeHostImpl::BeginCommit();
test_hooks_->BeginCommitOnThread(this);
@@ -114,20 +111,7 @@ class LayerTreeHostImplForTesting : public LayerTreeHostImpl {
test_hooks_->SwapBuffersCompleteOnThread(this);
}
- virtual void ActivatePendingTreeIfNeeded() OVERRIDE {
- if (!pending_tree())
- return;
-
- if (!test_hooks_->CanActivatePendingTreeIfNeeded(this))
- return;
-
- LayerTreeHostImpl::ActivatePendingTreeIfNeeded();
- }
-
virtual void ActivatePendingTree() OVERRIDE {
- if (!test_hooks_->CanActivatePendingTree(this))
- return;
-
test_hooks_->WillActivateTreeOnThread(this);
LayerTreeHostImpl::ActivatePendingTree();
DCHECK(!pending_tree());

Powered by Google App Engine
This is Rietveld 408576698