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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerBridge.java

Issue 2830843002: [Offline pages] Updates to background scheduling to use BTS (Closed)
Patch Set: Fixing the crash on NCN not being initialized Created 3 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: chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerBridge.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerBridge.java b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerBridge.java
index cf09c6a84d6fb23448d24a8ad44394174a95936f..d18f73df3be01fae2728893ab451a270538af04a 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerBridge.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerBridge.java
@@ -9,6 +9,8 @@ import org.chromium.base.ContextUtils;
import org.chromium.base.annotations.CalledByNative;
import org.chromium.base.annotations.JNINamespace;
+import java.util.concurrent.TimeUnit;
+
/**
* Provides Java scheduling support from native offlining code as
* well as JNI interface to tell native code to start processing
@@ -42,19 +44,18 @@ public class BackgroundSchedulerBridge {
@CalledByNative
private static void schedule(TriggerConditions triggerConditions) {
- BackgroundScheduler.getInstance(ContextUtils.getApplicationContext())
- .schedule(triggerConditions);
+ BackgroundScheduler.getInstance().schedule(triggerConditions);
}
@CalledByNative
private static void backupSchedule(TriggerConditions triggerConditions, long delayInSeconds) {
- BackgroundScheduler.getInstance(ContextUtils.getApplicationContext())
- .scheduleBackup(triggerConditions, delayInSeconds);
+ BackgroundScheduler.getInstance().scheduleBackup(
+ triggerConditions, TimeUnit.SECONDS.toMillis(delayInSeconds));
}
@CalledByNative
private static void unschedule() {
- BackgroundScheduler.getInstance(ContextUtils.getApplicationContext()).cancel();
+ BackgroundScheduler.getInstance().cancel();
}
@CalledByNative

Powered by Google App Engine
This is Rietveld 408576698