Created multi-process-friendly PowerMonitor interface.

base/power_monitor/power_monitor_source.cc

+// 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 accessor, even for friends.

[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