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

Unified Diff: content/browser/battery_status/battery_status_manager_default.cc

Issue 356873002: battery-status: Implement the battery-status API for chromeos. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 6 years, 5 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
Index: content/browser/battery_status/battery_status_manager_default.cc
diff --git a/content/browser/battery_status/battery_status_manager_default.cc b/content/browser/battery_status/battery_status_manager_default.cc
index a8d066bdefd14c0beee40e51de18682a8e1c96aa..cd037cf37c984e66d54b1d2a8d2fdf084e28f686 100644
--- a/content/browser/battery_status/battery_status_manager_default.cc
+++ b/content/browser/battery_status/battery_status_manager_default.cc
@@ -8,24 +8,33 @@
namespace content {
-BatteryStatusManager::BatteryStatusManager(
- const BatteryStatusService::BatteryUpdateCallback& callback)
- : callback_(callback) {
-}
+namespace {
-BatteryStatusManager::BatteryStatusManager() {
-}
+class BatteryStatusManagerDefault : public BatteryStatusManager {
+ public:
+ explicit BatteryStatusManagerDefault(
+ const BatteryStatusService::BatteryUpdateCallback& callback) {}
+ virtual ~BatteryStatusManagerDefault() {}
-BatteryStatusManager::~BatteryStatusManager() {
-}
+ private:
+ // BatteryStatusManager:
+ virtual bool StartListeningBatteryChange() OVERRIDE {
+ NOTIMPLEMENTED();
+ return false;
+ }
-bool BatteryStatusManager::StartListeningBatteryChange() {
- NOTIMPLEMENTED();
- return false;
-}
+ virtual void StopListeningBatteryChange() OVERRIDE { NOTIMPLEMENTED(); }
+
+ DISALLOW_COPY_AND_ASSIGN(BatteryStatusManagerDefault);
+};
+
+} // namespace
-void BatteryStatusManager::StopListeningBatteryChange() {
- NOTIMPLEMENTED();
+// static
+scoped_ptr<BatteryStatusManager> BatteryStatusManager::Create(
+ const BatteryStatusService::BatteryUpdateCallback& callback) {
+ return scoped_ptr<BatteryStatusManager>(
+ new BatteryStatusManagerDefault(callback));
}
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698