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

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

Issue 2066593003: Add a general purpose facility for letting background tasks wait until (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/android/java/src/org/chromium/chrome/browser/offlinepages/TaskExtrasPacker.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/TaskExtrasPacker.java b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/TaskExtrasPacker.java
index 44b7203dc9ad2ebc17f491f36720c1b0c515291d..42a37fd9ae1826956946bfb21d8192b804ed1a49 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/TaskExtrasPacker.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/TaskExtrasPacker.java
@@ -6,6 +6,8 @@ package org.chromium.chrome.browser.offlinepages;
import android.os.Bundle;
+import org.chromium.chrome.browser.ChromeBackgroundService;
+
/**
* Class to put our custom task information into the task bundle.
*/
@@ -16,6 +18,7 @@ public class TaskExtrasPacker {
/** Puts current time into the input bundle. */
public static void packTimeInBundle(Bundle bundle) {
bundle.putLong(SCHEDULED_TIME_TAG, System.currentTimeMillis());
+ bundle.putLong(ChromeBackgroundService.WAIT_NEEDED, 1);
dougarnett 2016/06/13 21:53:35 putBoolean
Pete Williamson 2016/06/13 23:23:10 There is no putBoolean in bundle. I kept this a l
dougarnett 2016/06/14 01:42:45 https://developer.android.com/reference/android/os
Pete Williamson 2016/06/15 00:25:19 Done.
}
/** Extracts the time we put into the bundle. */

Powered by Google App Engine
This is Rietveld 408576698