Disable scheduler deadline task on battery power in Windows

cc/scheduler/scheduler.cc

Index: cc/scheduler/scheduler.cc
diff --git a/cc/scheduler/scheduler.cc b/cc/scheduler/scheduler.cc
index 44d7d77ace95b2f839154854882e659ef51feda6..e5e06d8293cebfad4e162b07e9aca23a89a38ce6 100644
--- a/cc/scheduler/scheduler.cc
+++ b/cc/scheduler/scheduler.cc
@@ -117,9 +117,12 @@ Scheduler::Scheduler(
   background_frame_source_ =
       frame_sources_constructor->ConstructBackgroundFrameSource(this);
   frame_source_->AddSource(background_frame_source_);
+
+  SetupPowerMonitoring();
 }
 
 Scheduler::~Scheduler() {
+  TeardownPowerMonitoring();
 }
 
 base::TimeTicks Scheduler::Now() const {
@@ -131,6 +134,29 @@ base::TimeTicks Scheduler::Now() const {
   return now;
 }
 
+void Scheduler::SetupPowerMonitoring() {
+  if (settings_.prioritize_impl_latency_on_battery) {
+    base::PowerMonitor* power_monitor = client_->PowerMonitor();
+    DCHECK(power_monitor);
+    power_monitor->AddObserver(this);
+    state_machine_.SetImplLatencyTakesPriorityOnBattery(
+        power_monitor->IsOnBatteryPower());
+  }
+}
+
+void Scheduler::TeardownPowerMonitoring() {
+  if (settings_.prioritize_impl_latency_on_battery) {
+    base::PowerMonitor* power_monitor = client_->PowerMonitor();
+    DCHECK(power_monitor);
+    power_monitor->RemoveObserver(this);
+  }
+}
+
+void Scheduler::OnPowerStateChange(bool on_battery_power) {
+  DCHECK(settings_.prioritize_impl_latency_on_battery);
+  state_machine_.SetImplLatencyTakesPriorityOnBattery(on_battery_power);
+}
+
 void Scheduler::CommitVSyncParameters(base::TimeTicks timebase,
                                       base::TimeDelta interval) {
   // TODO(brianderson): We should not be receiving 0 intervals.
@@ -489,6 +515,7 @@ void Scheduler::BeginImplFrame(const BeginFrameArgs& args) {
   DCHECK_EQ(state_machine_.begin_impl_frame_state(),
             SchedulerStateMachine::BEGIN_IMPL_FRAME_STATE_IDLE);
   DCHECK(state_machine_.HasInitializedOutputSurface());
+  DCHECK(begin_impl_frame_deadline_task_.IsCancelled());

[sunnyps, 2014/10/13 23:59:43]

Will revert this too as it's not related to this change directly.

 
   advance_commit_state_task_.Cancel();
 
@@ -509,15 +536,19 @@ void Scheduler::BeginImplFrame(const BeginFrameArgs& args) {
   ProcessScheduledActions();
 
   state_machine_.OnBeginImplFrameDeadlinePending();
-  ScheduleBeginImplFrameDeadline(
-      AdjustedBeginImplFrameDeadline(args, draw_duration_estimate));
+
+  if (!settings_.using_synchronous_renderer_compositor) {
+    ScheduleBeginImplFrameDeadline(
+        AdjustedBeginImplFrameDeadline(args, draw_duration_estimate));
+  } else {
+    OnBeginImplFrameDeadline();
+  }
 }
 
 base::TimeTicks Scheduler::AdjustedBeginImplFrameDeadline(
     const BeginFrameArgs& args,
     base::TimeDelta draw_duration_estimate) const {
   if (settings_.using_synchronous_renderer_compositor) {
-    // The synchronous compositor needs to draw right away.

[mithro-old, 2014/10/13 23:25:57]

Is this being removed in your CL, or was this removed in a rebase?

[sunnyps, 2014/10/13 23:59:43]

I'll revert this and other changes around here that are not related to power
monitoring and make those a part of the retro frame CL.

     return base::TimeTicks();
   } else if (state_machine_.ShouldTriggerBeginImplFrameDeadlineEarly()) {
     // We are ready to draw a new active tree immediately.
@@ -541,15 +572,6 @@ base::TimeTicks Scheduler::AdjustedBeginImplFrameDeadline(
 void Scheduler::ScheduleBeginImplFrameDeadline(base::TimeTicks deadline) {
   TRACE_EVENT1(
       "cc", "Scheduler::ScheduleBeginImplFrameDeadline", "deadline", deadline);
-  if (settings_.using_synchronous_renderer_compositor) {

[mithro-old, 2014/10/13 23:25:57]

Is this being removed in your CL, or was this removed in a rebase?

[sunnyps, 2014/10/13 23:59:43]

Will revert this part.

-    // The synchronous renderer compositor has to make its GL calls
-    // within this call.
-    // TODO(brianderson): Have the OutputSurface initiate the deadline tasks
-    // so the sychronous renderer compositor can take advantage of splitting
-    // up the BeginImplFrame and deadline as well.
-    OnBeginImplFrameDeadline();
-    return;
-  }
   begin_impl_frame_deadline_task_.Cancel();
   begin_impl_frame_deadline_task_.Reset(begin_impl_frame_deadline_closure_);