cc: Fix infinite BeginFrame that Scheduler causes.

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 6446e8dcbb7a9b181676c62e84605a85ee1d2630..77cdcd5671b7c124d6ccc02a9510f787d4cb359f 100644
--- a/cc/scheduler/scheduler_state_machine.cc
+++ b/cc/scheduler/scheduler_state_machine.cc
@@ -365,11 +365,11 @@ bool SchedulerStateMachine::BeginFrameNeededToDrawByImplThread() const {
 
 bool SchedulerStateMachine::ProactiveBeginFrameWantedByImplThread() const {
   // We should proactively request a BeginFrame if a commit is pending.
-  if (needs_commit_ || needs_forced_commit_ ||
-      commit_state_ != COMMIT_STATE_IDLE)
-    return true;
-
-  return false;
+  // However, it is not needed to be proactive inside a BeginFrame
+  // because a commit is performed now.
+  return (needs_commit_ || needs_forced_commit_ ||
+      commit_state_ != COMMIT_STATE_IDLE) &&
+      !inside_begin_frame_;

[brianderson, 2013/06/24 20:12:31]

Can you see if this works instead? I would test it myself, but my tree is broken
right now.

if (!visible_ || output_surface_state_ != OUTPUT_SURFACE_ACTIVE)
  return false;

if (needs_commit_ || needs_forced_commit_ ||
    commit_state_ != COMMIT_STATE_IDLE)
  return true;

 }
 
 void SchedulerStateMachine::DidEnterBeginFrame(const BeginFrameArgs& args) {