cc: Control activation from the Scheduler

cc/scheduler/scheduler.h

Index: cc/scheduler/scheduler.h
diff --git a/cc/scheduler/scheduler.h b/cc/scheduler/scheduler.h
index c64dad0f23e24a74fa5f4fd902efa09fa576bf29..bf00ebb5b3cabeb550bd23fd17cc36fa51fd9e8f 100644
--- a/cc/scheduler/scheduler.h
+++ b/cc/scheduler/scheduler.h
@@ -41,7 +41,7 @@ class SchedulerClient {
   ScheduledActionDrawAndSwapForced() = 0;
   virtual void ScheduledActionCommit() = 0;
   virtual void ScheduledActionUpdateVisibleTiles() = 0;
-  virtual void ScheduledActionActivatePendingTreeIfNeeded() = 0;
+  virtual void ScheduledActionActivatePendingTree() = 0;
   virtual void ScheduledActionBeginOutputSurfaceCreation() = 0;
   virtual void ScheduledActionAcquireLayerTexturesForMainThread() = 0;
   virtual void DidAnticipatedDrawTimeChange(base::TimeTicks time) = 0;
@@ -67,6 +67,7 @@ class CC_EXPORT Scheduler {
 
   void SetVisible(bool visible);
   void SetCanDraw(bool can_draw);
+  void NotifyReadyToActivate();
   void SetHasPendingTree(bool has_pending_tree);

[enne (OOO), 2013/08/23 21:38:36]

I'm also not sure why the scheduler needs to be told explicitly about the
pending tree here when it's driving activation.  It's safe to assume that
activating = no pending tree and non-aborted commit with impl-side painting =
has pending tree.

I'm mostly balking at calculating it implicitly in
commit_results_in_pending_tree and then also getting it set explicitly here in
SetHasPendingTree.  It seems like it'd be cleaner to do one or the other but not
both.

[brianderson, 2013/08/23 23:11:52]

Ah, ok. I'm removing SetHasPendingTree now and it's definitely more clear.

 
   void SetNeedsCommit();