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

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

Issue 267893003: Battery Status API: add plumbing for multi-platform support. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 years, 7 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_android.cc
diff --git a/content/browser/battery_status/battery_status_manager_android.cc b/content/browser/battery_status/battery_status_manager_android.cc
index e7769ca2baa940011d64d3e256246f8d68889d62..99b92f4c6e2b5b492675fc6faa0102dc681d7a06 100644
--- a/content/browser/battery_status/battery_status_manager_android.cc
+++ b/content/browser/battery_status/battery_status_manager_android.cc
@@ -2,9 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "content/browser/battery_status/battery_status_manager_android.h"
-
-#include <string.h>
+#include "content/browser/battery_status/battery_status_manager.h"
#include "base/android/jni_android.h"
#include "jni/BatteryStatusManager_jni.h"
@@ -13,33 +11,39 @@ using base::android::AttachCurrentThread;
namespace content {
-BatteryStatusManagerAndroid::BatteryStatusManagerAndroid() {
+BatteryStatusManager::BatteryStatusManager(
+ const BatteryStatusUpdateCallback& callback) : callback_(callback) {
j_manager_.Reset(
Java_BatteryStatusManager_getInstance(
AttachCurrentThread(), base::android::GetApplicationContext()));
}
-BatteryStatusManagerAndroid::~BatteryStatusManagerAndroid() {
+BatteryStatusManager::~BatteryStatusManager() {
StopListeningBatteryChange();
}
-bool BatteryStatusManagerAndroid::Register(JNIEnv* env) {
+bool BatteryStatusManager::Register(JNIEnv* env) {
return RegisterNativesImpl(env);
}
-void BatteryStatusManagerAndroid::GotBatteryStatus(JNIEnv*, jobject,
- jboolean charging, jdouble chargingTime, jdouble dischargingTime,
+void BatteryStatusManager::GotBatteryStatus(JNIEnv*, jobject,
+ jboolean charging, jdouble charging_time, jdouble discharging_time,
jdouble level) {
- NOTIMPLEMENTED();
+ blink::WebBatteryStatus status;
+ status.charging = charging;
+ status.chargingTime = charging_time;
+ status.dischargingTime = discharging_time;
+ status.level = level;
+ callback_.Run(status);
}
-bool BatteryStatusManagerAndroid::StartListeningBatteryChange() {
+bool BatteryStatusManager::StartListeningBatteryChange() {
return Java_BatteryStatusManager_start(
AttachCurrentThread(), j_manager_.obj(),
reinterpret_cast<intptr_t>(this));
}
-void BatteryStatusManagerAndroid::StopListeningBatteryChange() {
+void BatteryStatusManager::StopListeningBatteryChange() {
Java_BatteryStatusManager_stop(
AttachCurrentThread(), j_manager_.obj());
}

Powered by Google App Engine
This is Rietveld 408576698