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

Unified Diff: base/power_monitor/power_monitor_android.cc

Issue 10959020: SystemMonitor refactoring: move power state monitor into a separate class called PowerMonitor (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Use PowerMonitor instead of SystemMonitor in XXXMain function Created 7 years, 9 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_android.cc
diff --git a/base/system_monitor/system_monitor_android.cc b/base/power_monitor/power_monitor_android.cc
similarity index 53%
rename from base/system_monitor/system_monitor_android.cc
rename to base/power_monitor/power_monitor_android.cc
index f29b7b0380279ac59461f02369142a687a24c297..786f07bd2d887f29c0e85d46deb16e15ace1542c 100644
--- a/base/system_monitor/system_monitor_android.cc
+++ b/base/power_monitor/power_monitor_android.cc
@@ -1,34 +1,41 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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/system_monitor/system_monitor.h"
+#include "base/power_monitor/power_monitor.h"
#include "jni/SystemMonitor_jni.h"
vandebo (ex-Chrome) 2013/03/19 23:36:04 SysMon->PowMon
Hongbo Min 2013/03/20 13:01:50 Done.
namespace base {
+// A helper function which is a friend of PowerMonitor.
+void ProcessPowerEventHelper(PowerMonitor::PowerEvent event) {
+ if (base::PowerMonitor::Get())
+ base::PowerMonitor::Get()->ProcessPowerEvent(event);
+}
+
namespace android {
// Native implementation of SystemMonitor.java.
void OnBatteryChargingChanged(JNIEnv* env, jclass clazz) {
- SystemMonitor::Get()->ProcessPowerMessage(SystemMonitor::POWER_STATE_EVENT);
+ ProcessPowerEventHelper(PowerMonitor::POWER_STATE_EVENT);
}
void OnMainActivityResumed(JNIEnv* env, jclass clazz) {
- SystemMonitor::Get()->ProcessPowerMessage(SystemMonitor::RESUME_EVENT);
+ ProcessPowerEventHelper(PowerMonitor::RESUME_EVENT);
}
void OnMainActivitySuspended(JNIEnv* env, jclass clazz) {
- SystemMonitor::Get()->ProcessPowerMessage(SystemMonitor::SUSPEND_EVENT);
+ ProcessPowerEventHelper(PowerMonitor::SUSPEND_EVENT);
}
} // namespace android
-bool SystemMonitor::IsBatteryPower() {
+bool PowerMonitor::IsBatteryPower() {
JNIEnv* env = base::android::AttachCurrentThread();
return base::android::Java_SystemMonitor_isBatteryPower(env);
}
+// TODO(hmin): Rename it to RegisterPowerMonitor.
vandebo (ex-Chrome) 2013/03/19 23:36:04 Do this in this CL.
Hongbo Min 2013/03/20 13:01:50 Done.
bool RegisterSystemMonitor(JNIEnv* env) {
return base::android::RegisterNativesImpl(env);
}

Powered by Google App Engine
This is Rietveld 408576698