Adding using protobuf based communication for the power information transfer.

chrome/browser/chromeos/dbus/power_manager_client.cc

 // 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.
 
 #include "chrome/browser/chromeos/dbus/power_manager_client.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/format_macros.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
 #include "base/stringprintf.h"
 #include "base/time.h"
 #include "base/timer.h"
+#include "chrome/browser/chromeos/dbus/power_supply_properties.pb.h"
 #include "chrome/browser/chromeos/login/screen_locker.h"
 #include "chrome/browser/chromeos/system/runtime_environment.h"
 #include "dbus/bus.h"
 #include "dbus/message.h"
 #include "dbus/object_proxy.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
 
 namespace chromeos {
 
 PowerSupplyStatus::PowerSupplyStatus()
@@ skipping 133 matching lines @@
         power_manager::kPowerManagerInterface,
         power_manager::kIncreaseScreenBrightness);
     power_manager_proxy_->CallMethod(
         &method_call,
         dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
         base::Bind(&PowerManagerClientImpl::OnIncreaseScreenBrightness,
                    weak_ptr_factory_.GetWeakPtr()));
   }
 
   virtual void RequestStatusUpdate() OVERRIDE {
-    dbus::MethodCall method_call(power_manager::kPowerManagerInterface,
+    dbus::MethodCall get_all_method_call(power_manager::kPowerManagerInterface,

[satorux1, 2012/02/07 21:16:49]

Do we still need to call the older method? I thought this one was replaced with
the protobuf version.

[rharrison, 2012/02/14 17:03:09]

Done.

                                  power_manager::kGetAllPropertiesMethod);

[satorux1, 2012/02/07 21:16:49]

nit: please align parameters vertically:

dbus::MethodCall get_all_method_call(
    power_manager::kPowerManagerInterface,
    power_manager::kGetAllPropertiesMethod);

[rharrison, 2012/02/14 17:03:09]

Done.

     power_manager_proxy_->CallMethod(
-        &method_call,
+        &get_all_method_call,
         dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
         base::Bind(&PowerManagerClientImpl::OnGetAllPropertiesMethod,
                    weak_ptr_factory_.GetWeakPtr()));
+
+    dbus::MethodCall get_power_method_call(
+        power_manager::kPowerManagerInterface,
+        power_manager::kGetPowerSupplyPropertiesMethod);
+    power_manager_proxy_->CallMethod(
+        &get_power_method_call,
+        dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
+        base::Bind(&PowerManagerClientImpl::OnGetPowerSupplyPropertiesMethod,
+                   weak_ptr_factory_.GetWeakPtr()));
+
   }
 
   // Requests restart of the system.
   virtual void RequestRestart() OVERRIDE {
     dbus::MethodCall method_call(power_manager::kPowerManagerInterface,
                                  power_manager::kRequestRestartMethod);
     power_manager_proxy_->CallMethod(
         &method_call,
         dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
         dbus::ObjectProxy::EmptyResponseCallback());
@@ skipping 150 matching lines @@
         !reader.PopBool(&status.battery_is_full)) {
       LOG(ERROR) << "Error reading response from powerd: "
                  << response->ToString();
       return;
     }
 
     VLOG(1) << "Power status: " << status.ToString();
     FOR_EACH_OBSERVER(Observer, observers_, PowerChanged(status));
   }
 
+
+  void OnGetPowerSupplyPropertiesMethod(dbus::Response* response) {
+    if (!response) {
+      LOG(ERROR) << "Error calling "
+                 << power_manager::kGetPowerSupplyPropertiesMethod;
+      return;
+    }
+
+    dbus::MessageReader reader(response);
+    PowerSupplyProperties protobuf;
+    reader.PopArrayOfBytesAsProto(&protobuf);
+
+    PowerSupplyStatus status;
+    status.line_power_on = protobuf.line_power_on();
+    status.battery_seconds_to_empty = protobuf.battery_time_to_empty();
+    status.battery_seconds_to_full = protobuf.battery_time_to_full();
+    status.battery_percentage = protobuf.battery_percentage();
+    status.battery_is_present = protobuf.battery_is_present();
+    status.battery_is_full = protobuf.battery_is_charged();
+
+    VLOG(1) << "Power status: " << status.ToString();
+    FOR_EACH_OBSERVER(Observer, observers_, PowerChanged(status));
+  }
+
   void OnGetIdleTime(const CalculateIdleTimeCallback& callback,
                      dbus::Response* response) {
     if (!response) {
       LOG(ERROR) << "Error calling " << power_manager::kGetIdleTime;
       return;
     }
     dbus::MessageReader reader(response);
     int64 idle_time_ms = 0;
     if (!reader.PopInt64(&idle_time_ms)) {
       LOG(ERROR) << "Error reading response from powerd: "
@@ skipping 145 matching lines @@
 
 PowerManagerClient* PowerManagerClient::Create(dbus::Bus* bus) {
   if (system::runtime_environment::IsRunningOnChromeOS()) {
     return new PowerManagerClientImpl(bus);
   } else {
     return new PowerManagerClientStubImpl();
   }
 }
 
 }  // namespace chromeos