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..7c98e335b062de2cca75b19411a246780ef94db7 100644
--- a/chromeos/dbus/power_manager_client.cc
+++ b/chromeos/dbus/power_manager_client.cc
@@ -246,6 +246,26 @@ class PowerManagerClientImpl : public PowerManagerClient {
         dbus::ObjectProxy::EmptyResponseCallback());
   }
 
+  void SetBacklightsForcedOff(bool is_off) override {
+    dbus::MethodCall method_call(power_manager::kPowerManagerInterface,
+                                 power_manager::kSetBacklightsForcedOffMethod);
+    dbus::MessageWriter writer(&method_call);
+    writer.AppendBool(is_off);

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

nit: this can be shortened to:

  dbus::MessageWriter(&method_call).AppendBool(forced_off);

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

Done.

+    power_manager_proxy_->CallMethod(
+        &method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
+        dbus::ObjectProxy::EmptyResponseCallback());
+  }
+
+  void GetBacklightsForcedOffState(
+      const GetBacklightsForcedOffStateCallback& 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::OnGetBacklightsForcedOffState,
+                   weak_ptr_factory_.GetWeakPtr(), callback));
+  }
+
   base::Closure GetSuspendReadinessCallback() override {
     DCHECK(OnOriginThread());
     DCHECK(suspend_is_pending_);
@@ -477,6 +497,22 @@ class PowerManagerClientImpl : public PowerManagerClient {
     callback.Run(percent);
   }
 
+  void OnGetBacklightsForcedOffState(
+      const GetBacklightsForcedOffStateCallback& 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))

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

nit: multi-line if bodies need curly brackets

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

Done.

+      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));