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

Unified Diff: cc/scheduler/scheduler_unittest.cc

Issue 2641033002: cc: Remove unused scheduler setting. (Closed)
Patch Set: Created 3 years, 11 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 | « cc/scheduler/scheduler_state_machine_unittest.cc ('k') | cc/trees/layer_tree_settings.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/scheduler/scheduler_unittest.cc
diff --git a/cc/scheduler/scheduler_unittest.cc b/cc/scheduler/scheduler_unittest.cc
index 64f4acd3eee5575f9ad1f09eb52296b0c986a8b4..f3bf54b89eb0b412472126725fd315fe5ac97e16 100644
--- a/cc/scheduler/scheduler_unittest.cc
+++ b/cc/scheduler/scheduler_unittest.cc
@@ -3183,34 +3183,6 @@ TEST_F(SchedulerTest, NoCompositorFrameSinkCreationWhileCommitPending) {
client_);
}
-TEST_F(SchedulerTest, CompositorFrameSinkCreationWhileCommitPending) {
- scheduler_settings_.abort_commit_before_compositor_frame_sink_creation =
- false;
- SetUpScheduler(THROTTLED_BFS);
-
- // SetNeedsBeginMainFrame should begin the frame.
- scheduler_->SetNeedsBeginMainFrame();
- client_->Reset();
- EXPECT_SCOPED(AdvanceFrame());
- EXPECT_ACTION("WillBeginImplFrame", client_, 0, 2);
- EXPECT_ACTION("ScheduledActionSendBeginMainFrame", client_, 1, 2);
-
- // Lose the CompositorFrameSink and trigger the deadline.
- client_->Reset();
- scheduler_->DidLoseCompositorFrameSink();
- EXPECT_TRUE(client_->IsInsideBeginImplFrame());
- EXPECT_NO_ACTION(client_);
-
- // The scheduler should trigger the CompositorFrameSink creation immediately
- // after
- // the begin_impl_frame_state_ is cleared.
- task_runner_->RunTasksWhile(client_->InsideBeginImplFrame(true));
- EXPECT_FALSE(client_->IsInsideBeginImplFrame());
- EXPECT_ACTION("ScheduledActionBeginCompositorFrameSinkCreation", client_, 0,
- 2);
- EXPECT_ACTION("SendBeginMainFrameNotExpectedSoon", client_, 1, 2);
-}
-
// The three letters appeneded to each version of this test mean the following:s
// tree_priority: B = both trees same priority; A = active tree priority;
// scroll_handler_state: H = affects scroll handler; N = does not affect scroll
« no previous file with comments | « cc/scheduler/scheduler_state_machine_unittest.cc ('k') | cc/trees/layer_tree_settings.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698