Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(685)

Unified Diff: chrome/browser/chromeos/cros/power_library.cc

Issue 8263003: cros: PowerLibrary virtual functions should be named using CamelCase. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix browser_tests Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/chromeos/cros/power_library.h ('k') | chrome/browser/chromeos/low_battery_observer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/cros/power_library.cc
diff --git a/chrome/browser/chromeos/cros/power_library.cc b/chrome/browser/chromeos/cros/power_library.cc
index 6ebb11df7d26e2fb8b8157e4af2a1b1e651c1d1e..acde536b9fdccfb3aa2412f8916cd9607841f29c 100644
--- a/chrome/browser/chromeos/cros/power_library.cc
+++ b/chrome/browser/chromeos/cros/power_library.cc
@@ -53,27 +53,27 @@ class PowerLibraryImpl : public PowerLibrary {
observers_.RemoveObserver(observer);
}
- virtual bool line_power_on() const OVERRIDE {
+ virtual bool IsLinePowerOn() const OVERRIDE {
return status_.line_power_on;
}
- virtual bool battery_fully_charged() const OVERRIDE {
+ virtual bool IsBatteryFullyCharged() const OVERRIDE {
return status_.battery_state == chromeos::BATTERY_STATE_FULLY_CHARGED;
}
- virtual double battery_percentage() const OVERRIDE {
+ virtual double GetBatteryPercentage() const OVERRIDE {
return status_.battery_percentage;
}
- virtual bool battery_is_present() const OVERRIDE {
+ virtual bool IsBatteryPresent() const OVERRIDE {
return status_.battery_is_present;
}
- virtual base::TimeDelta battery_time_to_empty() const OVERRIDE {
+ virtual base::TimeDelta GetBatteryTimeToEmpty() const OVERRIDE {
return base::TimeDelta::FromSeconds(status_.battery_time_to_empty);
}
- virtual base::TimeDelta battery_time_to_full() const OVERRIDE {
+ virtual base::TimeDelta GetBatteryTimeToFull() const OVERRIDE {
return base::TimeDelta::FromSeconds(status_.battery_time_to_full);
}
@@ -207,34 +207,34 @@ class PowerLibraryStubImpl : public PowerLibrary {
observers_.RemoveObserver(observer);
}
- virtual bool line_power_on() const OVERRIDE {
+ virtual bool IsLinePowerOn() const OVERRIDE {
return !discharging_;
}
- virtual bool battery_fully_charged() const OVERRIDE {
+ virtual bool IsBatteryFullyCharged() const OVERRIDE {
return battery_percentage_ == 100;
}
- virtual double battery_percentage() const OVERRIDE {
+ virtual double GetBatteryPercentage() const OVERRIDE {
return battery_percentage_;
}
- virtual bool battery_is_present() const OVERRIDE {
+ virtual bool IsBatteryPresent() const OVERRIDE {
return true;
}
- virtual base::TimeDelta battery_time_to_empty() const OVERRIDE {
+ virtual base::TimeDelta GetBatteryTimeToEmpty() const OVERRIDE {
if (battery_percentage_ == 0)
return base::TimeDelta::FromSeconds(1);
else
return (base::TimeDelta::FromHours(3) * battery_percentage_) / 100;
}
- virtual base::TimeDelta battery_time_to_full() const OVERRIDE {
+ virtual base::TimeDelta GetBatteryTimeToFull() const OVERRIDE {
if (battery_percentage_ == 100)
return base::TimeDelta::FromSeconds(1);
else
- return base::TimeDelta::FromHours(3) - battery_time_to_empty();
+ return base::TimeDelta::FromHours(3) - GetBatteryTimeToEmpty();
}
virtual void CalculateIdleTime(CalculateIdleTimeCallback* callback) OVERRIDE {
« no previous file with comments | « chrome/browser/chromeos/cros/power_library.h ('k') | chrome/browser/chromeos/low_battery_observer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698