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

Unified Diff: base/power_monitor/power_monitor_source.cc

Issue 17074009: Created multi-process-friendly PowerMonitor interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressing feedback from palmer and vandebo Created 7 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: base/power_monitor/power_monitor_source.cc
diff --git a/base/power_monitor/power_monitor_source.cc b/base/power_monitor/power_monitor_source.cc
new file mode 100644
index 0000000000000000000000000000000000000000..289dba9151617b70df6d97e40e2109ea66729a8b
--- /dev/null
+++ b/base/power_monitor/power_monitor_source.cc
@@ -0,0 +1,60 @@
+// Copyright 2013 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 "base/power_monitor/power_monitor_source.h"
+
+#include "base/power_monitor/power_monitor.h"
+
+namespace base {
+
+PowerMonitorSource::PowerMonitorSource()
+ : on_battery_power_(false),
+ suspended_(false) {
+}
+
+PowerMonitorSource::~PowerMonitorSource() {
+}
+
+bool PowerMonitorSource::IsOnBatteryPower() {
+ AutoLock auto_lock(lock_);
+ return on_battery_power_;
+}
+
+bool PowerMonitorSource::UpdateOnBatteryPower() {
+ AutoLock auto_lock(lock_);
+ bool old_on_battery_power = on_battery_power_;
+ on_battery_power_ = IsOnBatteryPowerImpl();
+ return old_on_battery_power != on_battery_power_;
+}
+
+void PowerMonitorSource::ProcessPowerEvent(PowerEvent event_id) {
+ PowerMonitor* monitor = PowerMonitor::Get();
+ if (!monitor)
+ return;
+
+ PowerMonitorSource* source = monitor->source_.get();
vandebo (ex-Chrome) 2013/07/10 22:37:34 I think the preference is to have a private access
bajones 2013/07/10 22:44:38 Done.
+
+ // Suppress duplicate notifications. Some platforms may
+ // send multiple notifications of the same event.
+ switch (event_id) {
+ case POWER_STATE_EVENT:
+ if (source->UpdateOnBatteryPower())
+ monitor->NotifyPowerStateChange(source->IsOnBatteryPower());
+ break;
+ case RESUME_EVENT:
+ if (source->suspended_) {
+ source->suspended_ = false;
+ monitor->NotifyResume();
+ }
+ break;
+ case SUSPEND_EVENT:
+ if (!source->suspended_) {
+ source->suspended_ = true;
+ monitor->NotifySuspend();
+ }
+ break;
+ }
+}
+
+} // namespace base

Powered by Google App Engine
This is Rietveld 408576698