cc: Remove background ticking from LTHI.

cc/scheduler/scheduler_state_machine_unittest.cc

 // Copyright 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/scheduler/scheduler_state_machine.h"
 
 #include "base/trace_event/trace_event.h"
 #include "cc/scheduler/scheduler.h"
 #include "cc/test/begin_frame_args_test.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 153 matching lines @@
   state.SetNeedsRedraw(true);
   state.SetNeedsAnimateForTest(false);
   EXPECT_TRUE(state.BeginFrameNeeded());
 
   // Don't background tick for needs_redraw.
   state.SetVisible(false);
   state.SetNeedsRedraw(true);
   state.SetNeedsAnimateForTest(false);
   EXPECT_FALSE(state.BeginFrameNeeded());
 
-  // Background tick for animations.
-  state.SetVisible(false);
-  state.SetNeedsRedraw(false);
-  state.SetNeedsAnimateForTest(true);
-  EXPECT_TRUE(state.BeginFrameNeeded());
-
   // Proactively request BeginFrames when commit is pending.
   state.SetVisible(true);
   state.SetNeedsRedraw(false);
   state.SetNeedsAnimateForTest(false);
   state.SetNeedsCommitForTest(true);
   EXPECT_TRUE(state.BeginFrameNeeded());
 
   // Don't request BeginFrames when commit is pending if
   // we are currently deferring commits.
   state.SetVisible(true);
@@ skipping 1388 matching lines @@
   state.NotifyReadyToCommit();
   EXPECT_ACTION_UPDATE_STATE(SchedulerStateMachine::ACTION_COMMIT);
 
   EXPECT_TRUE(state.PendingActivationsShouldBeForced());
   EXPECT_ACTION_UPDATE_STATE(SchedulerStateMachine::ACTION_ACTIVATE_SYNC_TREE);
 
   EXPECT_TRUE(state.PendingDrawsShouldBeAborted());
   EXPECT_ACTION_UPDATE_STATE(SchedulerStateMachine::ACTION_DRAW_AND_SWAP_ABORT);
 }
 
+TEST(SchedulerStateMachineTest, TestNoBeginFrameNeededWhenInvisible) {
+  SchedulerSettings default_scheduler_settings;
+  StateMachine state(default_scheduler_settings);
+  state.SetCanStart();
+  state.UpdateState(state.NextAction());
+  state.CreateAndInitializeOutputSurfaceWithActivatedCommit();
+  state.SetVisible(true);
+
+  EXPECT_FALSE(state.BeginFrameNeeded());
+  state.SetNeedsRedraw(true);
+  EXPECT_TRUE(state.BeginFrameNeeded());
+
+  state.SetVisible(false);
+  EXPECT_FALSE(state.BeginFrameNeeded());
+
+  state.SetVisible(true);
+  EXPECT_TRUE(state.BeginFrameNeeded());
+}
+
 TEST(SchedulerStateMachineTest, TestNoBeginMainFrameWhenInvisible) {
   SchedulerSettings default_scheduler_settings;
   StateMachine state(default_scheduler_settings);
   state.SetCanStart();
   state.UpdateState(state.NextAction());
   state.CreateAndInitializeOutputSurfaceWithActivatedCommit();
   state.SetVisible(false);
   state.SetNeedsCommit();
   EXPECT_ACTION(SchedulerStateMachine::ACTION_NONE);
+  EXPECT_FALSE(state.BeginFrameNeeded());
+
+  // When become visible again, the needs commit should still be pending.
+  state.SetVisible(true);
+  EXPECT_TRUE(state.BeginFrameNeeded());
+  state.OnBeginImplFrame();
+  EXPECT_ACTION_UPDATE_STATE(
+      SchedulerStateMachine::ACTION_SEND_BEGIN_MAIN_FRAME);
 }
 
 TEST(SchedulerStateMachineTest, TestFinishCommitWhenCommitInProgress) {
   SchedulerSettings default_scheduler_settings;
   StateMachine state(default_scheduler_settings);
   state.SetCanStart();
   state.UpdateState(state.NextAction());
   state.CreateAndInitializeOutputSurfaceWithActivatedCommit();
   state.SetVisible(false);
   state.SetCommitState(
@@ skipping 340 matching lines @@
   EXPECT_FALSE(state.ProactiveBeginFrameWanted());
   bool commit_has_no_updates = true;
   state.UpdateStateOnCommit(commit_has_no_updates);
   EXPECT_TRUE(state.ProactiveBeginFrameWanted());
   state.OnBeginImplFrame();
   EXPECT_FALSE(state.ProactiveBeginFrameWanted());
 }
 
 }  // namespace
 }  // namespace cc