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

Unified Diff: chrome/browser/android/offline_pages/background_scheduler_bridge.cc

Issue 2087633002: Defines TriggerConditions and plumbs up to BackgroundScheduler (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Exposed TriggerConditions object for test Created 4 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: chrome/browser/android/offline_pages/background_scheduler_bridge.cc
diff --git a/chrome/browser/android/offline_pages/background_scheduler_bridge.cc b/chrome/browser/android/offline_pages/background_scheduler_bridge.cc
index 4e4de65f8c45265913317e3c5e859db3bef98baa..6c9ceef632261d784acd56b73b15baab2f017d41 100644
--- a/chrome/browser/android/offline_pages/background_scheduler_bridge.cc
+++ b/chrome/browser/android/offline_pages/background_scheduler_bridge.cc
@@ -54,10 +54,13 @@ static jboolean StartProcessing(JNIEnv* env,
}
void BackgroundSchedulerBridge::Schedule(
- const TriggerCondition& trigger_condition) {
+ const TriggerConditions& trigger_conditions) {
JNIEnv* env = base::android::AttachCurrentThread();
- // TODO(dougarnett): pass trigger_condition.
- Java_BackgroundSchedulerBridge_schedule(env);
+ ScopedJavaLocalRef<jobject> j_conditions =
+ CreateTriggerConditions(env, trigger_conditions.require_power_connected,
+ trigger_conditions.minimum_battery_percentage,
+ trigger_conditions.require_unmetered_network);
+ Java_BackgroundSchedulerBridge_schedule(env, j_conditions.obj());
}
void BackgroundSchedulerBridge::Unschedule() {
@@ -65,6 +68,16 @@ void BackgroundSchedulerBridge::Unschedule() {
Java_BackgroundSchedulerBridge_unschedule(env);
}
+ScopedJavaLocalRef<jobject> BackgroundSchedulerBridge::CreateTriggerConditions(
+ JNIEnv* env,
+ bool require_power_connected,
+ int minimum_battery_percentage,
+ bool require_unmetered_network) const {
+ return Java_BackgroundSchedulerBridge_createTriggerConditions(
+ env, require_power_connected, minimum_battery_percentage,
+ require_unmetered_network);
+}
+
bool RegisterBackgroundSchedulerBridge(JNIEnv* env) {
return RegisterNativesImpl(env);
}

Powered by Google App Engine
This is Rietveld 408576698