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

chromeos/dbus/power_manager_client.cc

Index: chromeos/dbus/power_manager_client.cc
diff --git a/chromeos/dbus/power_manager_client.cc b/chromeos/dbus/power_manager_client.cc
index 3186f2ffefca09a4749dba19119694d173ae4b7e..d981f8350d00412e48cd675b4a04639724e0e7e8 100644
--- a/chromeos/dbus/power_manager_client.cc
+++ b/chromeos/dbus/power_manager_client.cc
@@ -246,6 +246,25 @@ class PowerManagerClientImpl : public PowerManagerClient {
         dbus::ObjectProxy::EmptyResponseCallback());
   }
 
+  void SetBacklightsForcedOff(bool forced_off) override {
+    dbus::MethodCall method_call(power_manager::kPowerManagerInterface,
+                                 power_manager::kSetBacklightsForcedOffMethod);
+    dbus::MessageWriter(&method_call).AppendBool(forced_off);
+    power_manager_proxy_->CallMethod(
+        &method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
+        dbus::ObjectProxy::EmptyResponseCallback());
+  }
+
+  void GetBacklightsForcedOff(
+      const GetBacklightsForcedOffCallback& callback) override {
+    dbus::MethodCall method_call(power_manager::kPowerManagerInterface,
+                                 power_manager::kGetBacklightsForcedOffMethod);
+    power_manager_proxy_->CallMethod(
+        &method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
+        base::Bind(&PowerManagerClientImpl::OnGetBacklightsForcedOff,
+                   weak_ptr_factory_.GetWeakPtr(), callback));
+  }
+
   base::Closure GetSuspendReadinessCallback() override {
     DCHECK(OnOriginThread());
     DCHECK(suspend_is_pending_);
@@ -471,12 +490,29 @@ class PowerManagerClientImpl : public PowerManagerClient {
     }
     dbus::MessageReader reader(response);
     double percent = 0.0;
-    if (!reader.PopDouble(&percent))
+    if (!reader.PopDouble(&percent)) {

[Daniel Erat, 2016/10/13 21:37:35]

ah, thanks for fixing this too :-)

       POWER_LOG(ERROR) << "Error reading response from powerd: "
                        << response->ToString();
+    }
     callback.Run(percent);
   }
 
+  void OnGetBacklightsForcedOff(const GetBacklightsForcedOffCallback& callback,
+                                dbus::Response* response) {
+    if (!response) {
+      POWER_LOG(ERROR) << "Error calling "
+                       << power_manager::kGetBacklightsForcedOffMethod;
+      return;
+    }
+    dbus::MessageReader reader(response);
+    bool state = false;
+    if (!reader.PopBool(&state)) {
+      POWER_LOG(ERROR) << "Error reading response from powerd: "
+                       << response->ToString();
+    }
+    callback.Run(state);
+  }
+
   void HandlePowerSupplyProperties(
       const power_manager::PowerSupplyProperties& proto) {
     FOR_EACH_OBSERVER(Observer, observers_, PowerChanged(proto));