cc: Always use SetNeedsBeginFrame to request the next BeginFrame

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 009cf4bdb59e61ecaf44ae268efc1359caf24a19..7b0ceca166023d5755f77592a40a4f97b0093fae 100644
--- a/cc/scheduler/scheduler_state_machine.cc
+++ b/cc/scheduler/scheduler_state_machine.cc
@@ -740,6 +740,11 @@ bool SchedulerStateMachine::ProactiveBeginFrameWantedByImplThread() const {
   if (has_pending_tree_)
     return true;
 
+  // If we just swapped, it's likely that we are going to produce another
+  // frame soon.
+  if (last_frame_number_swap_performed_ == current_frame_number_)

[enne (OOO), 2013/09/13 17:48:44]

Just as a sanity check, the bugfix here is that although the state machine is
swapping, maybe some output surface implementation doesn't swap, and thus the
next begin frame isn't unblocked.  By moving the check down a level (or two) to
the state machine, it knows when swaps have really happened and can request the
next begin frame without depending on the output surface to also unblock.

[brianderson, 2013/09/13 18:24:46]

Your understand is correct. The Scheduler can accept the next BeginFrame after a
swap request whether or not the swap actually occurred. ToT has the Scheduler
explicitly request the next BeginFrame when the OutputSurface's swap didn't
implicitly request the next BeginFrame. Having the Scheduler detect that
implicit request is too confusing and error prone, so this patch only uses the
explicit request.

This particular diff, however, is used to avoid negative glitches in
SetNeedsBeginFrame between two BeginFrames. Since we just drew, it's likely that
we don't think we need another BeginFrame for a fraction of a VSync until we get
another user-input or draw trigger.

A negative glitch is bad because it will propagate all the way to the Browser,
at which point the Browser might sample the SetNeedsBeginFrame value at an
inopportune time, delaying the BeginFrame for a VSync.

+    return true;
+
   return false;
 }