Increased battery time to empty sent by power manager stub impl

chrome/browser/chromeos/dbus/power_manager_client.cc

 // Copyright (c) 2011 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 "base/bind.h"
 #include "base/callback.h"
 #include "base/format_macros.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 258 matching lines @@
 
   // PowerManagerClient override.
   virtual void IncreaseScreenBrightness() OVERRIDE {
     VLOG(1) << "Requested to increase screen brightness";
   }
 
   virtual void RequestStatusUpdate() OVERRIDE {
     if (!timer_.IsRunning()) {
       timer_.Start(
           FROM_HERE,
-          base::TimeDelta::FromMilliseconds(100),
+          base::TimeDelta::FromMilliseconds(1000),
           this,
           &PowerManagerClientStubImpl::Update);
     } else {
       timer_.Stop();
     }
   }
 
   virtual void RequestRestart() OVERRIDE {}
 
   virtual void RequestShutdown() OVERRIDE {}
 
  private:
   void Update() {
     // We pause at 0 and 100% so that it's easier to check those conditions.
     if (pause_count_ > 1) {
       pause_count_--;
       return;
     }
 
     if (battery_percentage_ == 0 || battery_percentage_ == 100) {
       if (pause_count_) {
         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));
   }
 
   bool discharging_;
   int battery_percentage_;
   int pause_count_;
   ObserverList<Observer> observers_;
   base::RepeatingTimer<PowerManagerClientStubImpl> timer_;
 };
 
 PowerManagerClient::PowerManagerClient() {
 }
 
 PowerManagerClient::~PowerManagerClient() {
 }
 
 PowerManagerClient* PowerManagerClient::Create(dbus::Bus* bus) {
   if (system::runtime_environment::IsRunningOnChromeOS()) {
     return new PowerManagerClientImpl(bus);
   } else {
     return new PowerManagerClientStubImpl();
   }
 }
 
 }  // namespace chromeos