Make [S,G]getBacklightsForcedOff dbus API integrated to power_manager_client

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/callback.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "base/time/time.h"
 #include "chromeos/chromeos_export.h"
 #include "chromeos/dbus/dbus_client.h"
 #include "chromeos/dbus/dbus_client_implementation_type.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
 
 namespace power_manager {
 class PowerManagementPolicy;
 class PowerSupplyProperties;
 }
 
 namespace chromeos {
 
 // Callback used for getting the current screen brightness.  The param is in the
 // range [0.0, 100.0].
 typedef base::Callback<void(double)> GetScreenBrightnessPercentCallback;
 
+// Callback used for getting the current backlights forced off state.
+typedef base::Callback<void(bool)> GetBacklightsForcedOffStateCallback;
+
 // PowerManagerClient is used to communicate with the power manager.
 class CHROMEOS_EXPORT PowerManagerClient : public DBusClient {
  public:
   // Interface for observing changes from the power manager.
   class Observer {
    public:
     virtual ~Observer() {}
 
     // Called if the power manager process restarts.
     virtual void PowerManagerRestarted() {}
@@ skipping 137 matching lines @@
 
   // Tells powerd whether or not we are in a projecting mode.  This is used to
   // adjust idleness thresholds and derived, on this side, from the number of
   // video outputs attached.
   virtual void SetIsProjecting(bool is_projecting) = 0;
 
   // Tells powerd to change the power source to the given ID. An empty string
   // causes powerd to switch to using the battery on devices with type-C ports.
   virtual void SetPowerSource(const std::string& id) = 0;
 
+  // Forces the display and keyboard backlights (if present) to |is_off|.
+  virtual void SetBacklightsForcedOff(bool is_off) = 0;

[Daniel Erat, 2016/10/13 17:25:19]

nit: "forced_off" might be a tiny bit clearer than "is_off"

[Qiang(Joe) Xu, 2016/10/13 21:19:41]

Done.

+
+  // Gets the display and keyboard backlights (if present) forced off state.
+  virtual void GetBacklightsForcedOffState(

[Daniel Erat, 2016/10/13 17:25:19]

any reason not to call this (and its callback) just "GetBacklightsForcedOff" to
match the d-bus method?

[Qiang(Joe) Xu, 2016/10/13 21:19:41]

Changed to match dbus method.

+      const GetBacklightsForcedOffStateCallback& callback) = 0;
+
   // Returns a callback that can be called by an observer to report
   // readiness for suspend.  See Observer::SuspendImminent().
   virtual base::Closure GetSuspendReadinessCallback() = 0;
 
   // Returns the number of callbacks returned by GetSuspendReadinessCallback()
   // for the current suspend attempt but not yet called. Used by tests.
   virtual int GetNumPendingSuspendReadinessCallbacks() = 0;
 
   // Creates the instance.
   static PowerManagerClient* Create(DBusClientImplementationType type);
 
   ~PowerManagerClient() override;
 
  protected:
   // Needs to call DBusClient::Init().
   friend class PowerManagerClientTest;
 
   // Create() should be used instead.
   PowerManagerClient();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(PowerManagerClient);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_POWER_MANAGER_CLIENT_H_