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

Unified Diff: cc/test/layer_tree_test.cc

Issue 2188093002: cc: Complete swap promise for aborted commit after pending activation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix compile error Created 4 years, 5 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
« no previous file with comments | « no previous file | cc/trees/channel_main.h » ('j') | cc/trees/layer_tree_impl.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/test/layer_tree_test.cc
diff --git a/cc/test/layer_tree_test.cc b/cc/test/layer_tree_test.cc
index 49d4a70f5514022309333aae771db3a9817a3c3a..97c22f2c065f54f218166ae4251cc04f2bc031cc 100644
--- a/cc/test/layer_tree_test.cc
+++ b/cc/test/layer_tree_test.cc
@@ -147,8 +147,10 @@ class LayerTreeHostImplForTesting : public LayerTreeHostImpl {
test_hooks_->DidFinishImplFrameOnThread(this);
}
- void BeginMainFrameAborted(CommitEarlyOutReason reason) override {
- LayerTreeHostImpl::BeginMainFrameAborted(reason);
+ void BeginMainFrameAborted(
+ CommitEarlyOutReason reason,
+ std::vector<std::unique_ptr<SwapPromise>> swap_promises) override {
+ LayerTreeHostImpl::BeginMainFrameAborted(reason, std::move(swap_promises));
test_hooks_->BeginMainFrameAbortedOnThread(this, reason);
}
@@ -288,9 +290,7 @@ class LayerTreeHostClientForTesting : public LayerTreeHostClient,
test_hooks_->BeginMainFrame(args);
}
- void UpdateLayerTreeHost() override {
- test_hooks_->UpdateLayerTreeHost();
- }
+ void UpdateLayerTreeHost() override { test_hooks_->UpdateLayerTreeHost(); }
void ApplyViewportDeltas(const gfx::Vector2dF& inner_delta,
const gfx::Vector2dF& outer_delta,
@@ -505,8 +505,7 @@ void LayerTreeTest::EndTest() {
void LayerTreeTest::EndTestAfterDelayMs(int delay_milliseconds) {
main_task_runner_->PostDelayedTask(
- FROM_HERE,
- base::Bind(&LayerTreeTest::EndTest, main_thread_weak_ptr_),
+ FROM_HERE, base::Bind(&LayerTreeTest::EndTest, main_thread_weak_ptr_),
base::TimeDelta::FromMilliseconds(delay_milliseconds));
}
@@ -539,9 +538,8 @@ void LayerTreeTest::PostAddLongAnimationToMainThreadPlayer(
void LayerTreeTest::PostSetDeferCommitsToMainThread(bool defer_commits) {
main_task_runner_->PostTask(
- FROM_HERE,
- base::Bind(&LayerTreeTest::DispatchSetDeferCommits,
- main_thread_weak_ptr_, defer_commits));
+ FROM_HERE, base::Bind(&LayerTreeTest::DispatchSetDeferCommits,
+ main_thread_weak_ptr_, defer_commits));
}
void LayerTreeTest::PostSetNeedsCommitToMainThread() {
@@ -552,9 +550,8 @@ void LayerTreeTest::PostSetNeedsCommitToMainThread() {
void LayerTreeTest::PostSetNeedsUpdateLayersToMainThread() {
main_task_runner_->PostTask(
- FROM_HERE,
- base::Bind(&LayerTreeTest::DispatchSetNeedsUpdateLayers,
- main_thread_weak_ptr_));
+ FROM_HERE, base::Bind(&LayerTreeTest::DispatchSetNeedsUpdateLayers,
+ main_thread_weak_ptr_));
}
void LayerTreeTest::PostSetNeedsRedrawToMainThread() {
@@ -566,31 +563,26 @@ void LayerTreeTest::PostSetNeedsRedrawToMainThread() {
void LayerTreeTest::PostSetNeedsRedrawRectToMainThread(
const gfx::Rect& damage_rect) {
main_task_runner_->PostTask(
- FROM_HERE,
- base::Bind(&LayerTreeTest::DispatchSetNeedsRedrawRect,
- main_thread_weak_ptr_,
- damage_rect));
+ FROM_HERE, base::Bind(&LayerTreeTest::DispatchSetNeedsRedrawRect,
+ main_thread_weak_ptr_, damage_rect));
}
void LayerTreeTest::PostSetVisibleToMainThread(bool visible) {
- main_task_runner_->PostTask(
- FROM_HERE,
- base::Bind(
- &LayerTreeTest::DispatchSetVisible, main_thread_weak_ptr_, visible));
+ main_task_runner_->PostTask(FROM_HERE,
+ base::Bind(&LayerTreeTest::DispatchSetVisible,
+ main_thread_weak_ptr_, visible));
}
void LayerTreeTest::PostSetNextCommitForcesRedrawToMainThread() {
main_task_runner_->PostTask(
- FROM_HERE,
- base::Bind(&LayerTreeTest::DispatchSetNextCommitForcesRedraw,
- main_thread_weak_ptr_));
+ FROM_HERE, base::Bind(&LayerTreeTest::DispatchSetNextCommitForcesRedraw,
+ main_thread_weak_ptr_));
}
void LayerTreeTest::PostCompositeImmediatelyToMainThread() {
main_task_runner_->PostTask(
- FROM_HERE,
- base::Bind(&LayerTreeTest::DispatchCompositeImmediately,
- main_thread_weak_ptr_));
+ FROM_HERE, base::Bind(&LayerTreeTest::DispatchCompositeImmediately,
+ main_thread_weak_ptr_));
}
void LayerTreeTest::PostNextCommitWaitsForActivationToMainThread() {
« no previous file with comments | « no previous file | cc/trees/channel_main.h » ('j') | cc/trees/layer_tree_impl.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698