Increased battery time to empty sent by power manager stub impl

chrome/browser/chromeos/dbus/power_manager_client.cc

Index: chrome/browser/chromeos/dbus/power_manager_client.cc
diff --git a/chrome/browser/chromeos/dbus/power_manager_client.cc b/chrome/browser/chromeos/dbus/power_manager_client.cc
index dc9761d64cc6623ba667637f98eaf099ba5efada..9be6cb50d5fd66bdc1923abffa9d52a6f65e7045 100644
--- a/chrome/browser/chromeos/dbus/power_manager_client.cc
+++ b/chrome/browser/chromeos/dbus/power_manager_client.cc
@@ -276,7 +276,7 @@ class PowerManagerClientStubImpl : public PowerManagerClient {
     if (!timer_.IsRunning()) {
       timer_.Start(
           FROM_HERE,
-          base::TimeDelta::FromMilliseconds(100),
+          base::TimeDelta::FromMilliseconds(1000),
           this,
           &PowerManagerClientStubImpl::Update);
     } else {
@@ -301,20 +301,23 @@ class PowerManagerClientStubImpl : public PowerManagerClient {
         pause_count_ = 0;
         discharging_ = !discharging_;
       } else {
-        pause_count_ = 20;
+        pause_count_ = 2;

[satorux1, 2011/11/14 23:46:57]

Not your fault, but this logic looks rather awkward. You set 2 here, but we
decrement this counter only once at the beginning of the function, right? Adding
a comment like below may make it clear.

  // Pause twice (i.e. skip updating the menu), including the current call to
this function.
  pause_count_ = 2;

         return;
       }
     }
     battery_percentage_ += (discharging_ ? -1 : 1);
 
+    const int kSecondsToEmptyFullBattery(3 * 60 * 60); // 3 hours.

[satorux1, 2011/11/14 23:46:57]

two spaces before // per our style guide.

+
     PowerSupplyStatus status;
     status.line_power_on = !discharging_;
     status.battery_is_present = true;
     status.battery_percentage = battery_percentage_;
     status.battery_seconds_to_empty =
-        std::max(1, battery_percentage_ * 180 / 100);
+        std::max(1, battery_percentage_ * kSecondsToEmptyFullBattery / 100);
     status.battery_seconds_to_full =
-        std::max(static_cast<int64>(1), 180 - status.battery_seconds_to_empty);
+        std::max(static_cast<int64>(1),
+                 kSecondsToEmptyFullBattery - status.battery_seconds_to_empty);
 
     FOR_EACH_OBSERVER(Observer, observers_, PowerChanged(status));
   }