[Android] BackgroundTaskScheduler reschedule implementation

components/background_task_scheduler/android/java/src/org/chromium/components/background_task_scheduler/BackgroundTaskScheduler.java

Index: components/background_task_scheduler/android/java/src/org/chromium/components/background_task_scheduler/BackgroundTaskScheduler.java
diff --git a/components/background_task_scheduler/android/java/src/org/chromium/components/background_task_scheduler/BackgroundTaskScheduler.java b/components/background_task_scheduler/android/java/src/org/chromium/components/background_task_scheduler/BackgroundTaskScheduler.java
index 31f087ff96c2a6a824b0a94990760e86372e0a7c..d7570fca637dd1365ba0b07f4f7fea13f1c0f01c 100644
--- a/components/background_task_scheduler/android/java/src/org/chromium/components/background_task_scheduler/BackgroundTaskScheduler.java
+++ b/components/background_task_scheduler/android/java/src/org/chromium/components/background_task_scheduler/BackgroundTaskScheduler.java
@@ -13,6 +13,7 @@ import org.chromium.base.Log;
 import org.chromium.base.ThreadUtils;
 
 import java.lang.reflect.Constructor;
+import java.util.Set;
 
 /**
  * A BackgroundTaskScheduler which is used to schedule jobs that run in the background.
@@ -90,4 +91,18 @@ public class BackgroundTaskScheduler {
         BackgroundTaskSchedulerPrefs.removeScheduledTask(taskId);
         mSchedulerDelegate.cancel(context, taskId);
     }
+
+    public void reschedule(Context context) {
+        Set<String> scheduledTasksClassNames = BackgroundTaskSchedulerPrefs.getScheduledTasks();
+        BackgroundTaskSchedulerPrefs.removeAllTasks();

[nyquist, 2017/04/18 22:33:03]

So, conceptually we could do this before or after all the tasks, or before/after
each individual class.
You chose the before-all version, which is fine, but could you add a short
comment as to why? For example mention that if any of the tasks make us crash,
we won't crash repeatedly, etc., or whatever you thought process was behind
this.

[fgorski, 2017/04/20 19:09:58]

I did it to avoid updating the prefs entry from both directions (schedule and
here).
I can do one of the following:
* remove-all as here (upside: we start fresh, downside: if early task reschedule
crashes us, we lose all tasks)
* remove one task before: (upside: we don't lose tasks by crash of some other
task... downside: we update the prefs a lot...)
* remove one task after: (won't work as we will remove entry for rescheduled
task)
* don't remove tasks and make sure that next scheduling does not write if
already there (wont work if the task decides not to reschedule itself... it will
be pending there)

WDYT?

[nyquist, 2017/04/25 20:44:05]

I think what we have now is fine.

+        for (String className : scheduledTasksClassNames) {
+            BackgroundTask task = getBackgroundTaskFromClassName(className);
+            if (task == null) {
+                Log.w(TAG, "Cannot reschedule task for: " + className);
+                continue;
+            }
+
+            task.reschedule(context);
+        }
+    }
 }