Index: cc/scheduler/scheduler_state_machine_unittest.cc |
diff --git a/cc/scheduler/scheduler_state_machine_unittest.cc b/cc/scheduler/scheduler_state_machine_unittest.cc |
index 7036459794ffd34cc16364b34fba0d55cd65b478..b3bb6320655b780f062c0aeef783fa782382bc88 100644 |
--- a/cc/scheduler/scheduler_state_machine_unittest.cc |
+++ b/cc/scheduler/scheduler_state_machine_unittest.cc |
@@ -718,7 +718,7 @@ TEST(SchedulerStateMachineTest, TestGoesInvisibleBeforeFinishCommit) { |
// Become invisible and abort the main thread's begin frame. |
state.SetVisible(false); |
- state.BeginFrameAbortedByMainThread(); |
+ state.BeginFrameAbortedByMainThread(false); |
// We should now be back in the idle state as if we didn't start a frame at |
// all. |
@@ -728,6 +728,15 @@ TEST(SchedulerStateMachineTest, TestGoesInvisibleBeforeFinishCommit) { |
// Become visible again. |
state.SetVisible(true); |
+ // Because we have aborted on this frame, don't start another frame yet. |
+ EXPECT_EQ(SchedulerStateMachine::COMMIT_STATE_IDLE, state.CommitState()); |
+ EXPECT_EQ(SchedulerStateMachine::ACTION_NONE, state.NextAction()); |
+ |
+ // Start a new frame. |
+ state.DidEnterBeginFrame(BeginFrameArgs::CreateForTesting()); |
+ EXPECT_EQ(SchedulerStateMachine::ACTION_NONE, state.NextAction()); |
+ state.DidLeaveBeginFrame(); |
+ |
// We should be beginning a frame now. |
EXPECT_EQ(SchedulerStateMachine::COMMIT_STATE_IDLE, state.CommitState()); |
EXPECT_EQ(SchedulerStateMachine::ACTION_SEND_BEGIN_FRAME_TO_MAIN_THREAD, |
@@ -1192,7 +1201,7 @@ TEST(SchedulerStateMachineTest, |
// Become invisible and abort the main thread's begin frame. |
state.SetVisible(false); |
- state.BeginFrameAbortedByMainThread(); |
+ state.BeginFrameAbortedByMainThread(false); |
brianderson
2013/07/13 01:26:42
Should we add a test for BeginFrameAbortedByMainTh
enne (OOO)
2013/07/15 22:23:28
Oh, quite right, done.
|
// Should be back in the idle state, but needing a commit. |
EXPECT_EQ(SchedulerStateMachine::COMMIT_STATE_IDLE, state.CommitState()); |