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

Unified Diff: base/power_monitor/power_monitor_win.cc

Issue 17074009: Created multi-process-friendly PowerMonitor interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressing Feedback Created 7 years, 6 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_win.cc
diff --git a/base/power_monitor/power_monitor_win.cc b/base/power_monitor/power_monitor_win.cc
index 156ee2b24c04323c66a69620200627e3cff4a6e9..fd516c54689e9015e17dc688decd3d962bc1cdee 100644
--- a/base/power_monitor/power_monitor_win.cc
+++ b/base/power_monitor/power_monitor_win.cc
@@ -3,7 +3,7 @@
// found in the LICENSE file.
#include "base/power_monitor/power_monitor.h"
-
+#include "base/power_monitor/power_monitor_device_source.h"
#include "base/win/wrapped_window_proc.h"
namespace base {
@@ -16,7 +16,7 @@ const wchar_t kWindowClassName[] = L"Base_PowerMessageWindow";
// Function to query the system to see if it is currently running on
// battery power. Returns true if running on battery.
-bool PowerMonitor::IsBatteryPower() {
+bool PowerMonitorDeviceSource::IsBatteryPower() {
Ken Russell (switch to Gerrit) 2013/07/02 09:05:15 power_monitor_device_source_win.cc?
SYSTEM_POWER_STATUS status;
if (!GetSystemPowerStatus(&status)) {
DLOG_GETLASTERROR(ERROR) << "GetSystemPowerStatus failed";
@@ -25,7 +25,7 @@ bool PowerMonitor::IsBatteryPower() {
return (status.ACLineStatus == 0);
}
-PowerMonitor::PowerMessageWindow::PowerMessageWindow()
+PowerMonitorDeviceSource::PowerMessageWindow::PowerMessageWindow()
: instance_(NULL), message_hwnd_(NULL) {
if (MessageLoop::current()->type() != MessageLoop::TYPE_UI) {
// Creating this window in (e.g.) a renderer inhibits shutdown on Windows.
@@ -38,7 +38,7 @@ PowerMonitor::PowerMessageWindow::PowerMessageWindow()
base::win::InitializeWindowClass(
kWindowClassName,
&base::win::WrappedWindowProc<
- PowerMonitor::PowerMessageWindow::WndProcThunk>,
+ PowerMonitorDeviceSource::PowerMessageWindow::WndProcThunk>,
0, 0, 0, NULL, NULL, NULL, NULL, NULL,
&window_class);
instance_ = window_class.hInstance;
@@ -51,29 +51,30 @@ PowerMonitor::PowerMessageWindow::PowerMessageWindow()
reinterpret_cast<LONG_PTR>(this));
}
-PowerMonitor::PowerMessageWindow::~PowerMessageWindow() {
+PowerMonitorDeviceSource::PowerMessageWindow::~PowerMessageWindow() {
if (message_hwnd_) {
DestroyWindow(message_hwnd_);
UnregisterClass(kWindowClassName, instance_);
}
}
-void PowerMonitor::PowerMessageWindow::ProcessWmPowerBroadcastMessage(
+void
+PowerMonitorDeviceSource::PowerMessageWindow::ProcessWmPowerBroadcastMessage(
int event_id) {
- PowerMonitor::PowerEvent power_event;
+ PowerMonitorSource::PowerEvent power_event;
switch (event_id) {
case PBT_APMPOWERSTATUSCHANGE: // The power status changed.
- power_event = PowerMonitor::POWER_STATE_EVENT;
+ power_event = PowerMonitorSource::POWER_STATE_EVENT;
break;
case PBT_APMRESUMEAUTOMATIC: // Resume from suspend.
//case PBT_APMRESUMESUSPEND: // User-initiated resume from suspend.
// We don't notify for this latter event
// because if it occurs it is always sent as a
// second event after PBT_APMRESUMEAUTOMATIC.
- power_event = PowerMonitor::RESUME_EVENT;
+ power_event = PowerMonitorSource::RESUME_EVENT;
break;
case PBT_APMSUSPEND: // System has been suspended.
- power_event = PowerMonitor::SUSPEND_EVENT;
+ power_event = PowerMonitorSource::SUSPEND_EVENT;
break;
default:
return;
@@ -87,10 +88,10 @@ void PowerMonitor::PowerMessageWindow::ProcessWmPowerBroadcastMessage(
// PBT_POWERSETTINGCHANGE - user changed the power settings.
}
- PowerMonitor::Get()->ProcessPowerEvent(power_event);
+ PowerMonitor::Get()->Source()->ProcessPowerEvent(power_event);
}
-LRESULT CALLBACK PowerMonitor::PowerMessageWindow::WndProc(
+LRESULT CALLBACK PowerMonitorDeviceSource::PowerMessageWindow::WndProc(
HWND hwnd, UINT message, WPARAM wparam, LPARAM lparam) {
switch (message) {
case WM_POWERBROADCAST: {
@@ -105,10 +106,10 @@ LRESULT CALLBACK PowerMonitor::PowerMessageWindow::WndProc(
}
// static
-LRESULT CALLBACK PowerMonitor::PowerMessageWindow::WndProcThunk(
+LRESULT CALLBACK PowerMonitorDeviceSource::PowerMessageWindow::WndProcThunk(
HWND hwnd, UINT message, WPARAM wparam, LPARAM lparam) {
- PowerMonitor::PowerMessageWindow* message_hwnd =
- reinterpret_cast<PowerMonitor::PowerMessageWindow*>(
+ PowerMonitorDeviceSource::PowerMessageWindow* message_hwnd =
+ reinterpret_cast<PowerMonitorDeviceSource::PowerMessageWindow*>(
GetWindowLongPtr(hwnd, GWLP_USERDATA));
if (message_hwnd)
return message_hwnd->WndProc(hwnd, message, wparam, lparam);

Powered by Google App Engine
This is Rietveld 408576698