cc: Simplify SetNeedsBeginFrames code in scheduler.

cc/scheduler/scheduler_state_machine.cc

Index: cc/scheduler/scheduler_state_machine.cc
diff --git a/cc/scheduler/scheduler_state_machine.cc b/cc/scheduler/scheduler_state_machine.cc
index 189387cf12856f1fc348c605a976d1422aaf705c..5b5b386adb506c21f5761fb2bdf3252f831db92c 100644
--- a/cc/scheduler/scheduler_state_machine.cc
+++ b/cc/scheduler/scheduler_state_machine.cc
@@ -720,29 +720,6 @@ bool SchedulerStateMachine::BeginFrameNeeded() const {
   return BeginFrameNeededToAnimateOrDraw();
 }
 
-bool SchedulerStateMachine::ShouldSetNeedsBeginFrames(
-    bool frame_source_needs_begin_frames) const {
-  bool needs_begin_frame = BeginFrameNeeded();
-
-  // Never call SetNeedsBeginFrames if the frame source has the right value.
-  if (needs_begin_frame == frame_source_needs_begin_frames)
-    return false;
-
-  // Always request the BeginFrame immediately if it's needed.
-  if (needs_begin_frame)
-    return true;
-
-  // Stop requesting BeginFrames after a deadline.
-  if (begin_impl_frame_state_ == BEGIN_IMPL_FRAME_STATE_INSIDE_DEADLINE)
-    return true;
-
-  // Stop requesting BeginFrames immediately when output surface is lost.
-  if (!HasInitializedOutputSurface())

[brianderson, 2015/03/19 01:25:54]

This condition isn't taken into account in the code ported to the Scheduler, but
we'll trigger the deadline immediately in this case and so it shouldn't matter.
Is that true?


... nm. I see that your test changes capture this change in behavior and the
tests look good to me.

[sunnyps, 2015/03/19 01:31:53]

Yes, the new way of calling SetNeedsBeginFrames(false) is more general than what
we had before.

Before we would call SetNBF(false) when the output surface was lost. I'm pretty
sure this was done because the output surface can be lost in between BeginFrames
(idle state).

Now we call SetBNF(false) if we're idle.

-    return true;
-
-  return false;
-}
-
 bool SchedulerStateMachine::ShouldPollForAnticipatedDrawTriggers() const {
   // ShouldPollForAnticipatedDrawTriggers is what we use in place of
   // ProactiveBeginFrameWanted when we are using the synchronous