Add dark resume methods to PowerManagerClient

chromeos/dbus/power_manager_client.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROMEOS_DBUS_POWER_MANAGER_CLIENT_H_
 #define CHROMEOS_DBUS_POWER_MANAGER_CLIENT_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 53 matching lines @@
     // PowerManagerClient::GetSuspendReadinessCallback() may be called from
     // within SuspendImminent().  The returned callback must be called once
     // the observer is ready for suspend.
     virtual void SuspendImminent() {}
 
     // Called when a suspend attempt (previously announced via
     // SuspendImminent()) has completed. The system may not have actually
     // suspended (if e.g. the user canceled the suspend attempt).
     virtual void SuspendDone(const base::TimeDelta& sleep_duration) {}
 
+    // Called when the system is about to resuspend from a dark resume.  Like
+    // SuspendImminent(), the suspend will be deferred until all observers have
+    // finished running and those observers that wish to asynchromously delay

[Daniel Erat, 2014/08/15 23:49:31]

nit: s/asynchromously/asynchronously/

+    // the suspend should call PowerManagerClient::GetSuspendReadinessCallback()

[Daniel Erat, 2014/08/15 23:49:31]

sharing GetSuspendReadinessCallback() for this probably isn't the right
approach; i'm concerned that some observers that are just interested in normal
suspends will fail to call this before dark suspends. adding a new
GetDarkSuspendReadinessCallback() method is probably the best approach.

[Chirantan Ekbote, 2014/08/18 17:49:08]

I have no problem with adding a new GetDarkSuspendReadinessCallback() method but
I'm not sure I understand what you mean here.  Observers that only care about
normal suspends should not find out about dark suspends since they presumably
will not implement the DarkSuspendImminent() method.

+    // from within this method.  The returned callback should be run once the
+    // observer is ready for suspend.
+    virtual void DarkSuspendImminent() {}
+
     // Called when the power button is pressed or released.
     virtual void PowerButtonEventReceived(bool down,
                                           const base::TimeTicks& timestamp) {}
 
     // Called when the device's lid is opened or closed.
     virtual void LidEventReceived(bool open,
                                   const base::TimeTicks& timestamp) {}
 
     // Called when the idle action will be performed after
     // |time_until_idle_action|.
@@ skipping 76 matching lines @@
   // Create() should be used instead.
   PowerManagerClient();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(PowerManagerClient);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_POWER_MANAGER_CLIENT_H_