Add a general purpose facility for letting background tasks wait until

chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundOfflinerTask.java

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.offlinepages;
 
 import android.content.Context;
 import android.os.Bundle;
 
 import org.chromium.base.Callback;
+import org.chromium.base.Log;
 import org.chromium.base.VisibleForTesting;
+import org.chromium.chrome.browser.ChromeBackgroundServiceWaiter;
 import org.chromium.chrome.browser.offlinepages.interfaces.BackgroundSchedulerProcessor;
 
 /**
  * Handles servicing of background offlining requests coming via the GcmNetworkManager.
  */
 public class BackgroundOfflinerTask {
+    private static final String TAG = "BGOfflinerTask";
 
     public BackgroundOfflinerTask(BackgroundSchedulerProcessor bridge) {
         mBridge = bridge;
     }
 
     private final BackgroundSchedulerProcessor mBridge;
 
     /**
      * Triggers processing of background offlining requests.  This is called when
      * system conditions are appropriate for background offlining, typically from the
      * GcmTaskService onRunTask() method.  In response, we will start the
      * task processing by passing the call along to the C++ RequestCoordinator.
      * Also starts UMA collection.
      *
      * @returns true for success
      */
-    public boolean startBackgroundRequests(Context context, Bundle bundle) {
+    public boolean startBackgroundRequests(Context context, Bundle bundle,
+                                           ChromeBackgroundServiceWaiter waiter) {
+        mWaiter = waiter;
         processBackgroundRequests(bundle);
 
         // Gather UMA data to measure how often the user's machine is amenable to background
         // loading when we wake to do a task.
         long taskScheduledTimeMillis = TaskExtrasPacker.unpackTimeFromBundle(bundle);
         OfflinePageUtils.recordWakeupUMA(context, taskScheduledTimeMillis);
 
         return true;
     }
 
@@ skipping 12 matching lines @@
 
         // TODO(petewil): Decode the TriggerConditions from the bundle.
 
         Callback<Boolean> callback = new Callback<Boolean>() {
             /**
              * Callback function which indicates completion of background work.
              * @param result - true if work was actually done (used for UMA).
              */
             @Override
             public void onResult(Boolean result) {
-                // TODO(petewil): Release the wake lock.
+                // Release the wake lock.
+                Log.d(TAG, "onProcessingDone");
+                mWaiter.onWaitDone();
             }
         };
 
         // Pass the activation on to the bridge to the C++ RequestCoordinator.
         mBridge.startProcessing(callback);
     }
+
+    private ChromeBackgroundServiceWaiter mWaiter;

[dougarnett, 2016/06/13 21:53:35]

members before methods

[Pete Williamson, 2016/06/13 23:23:10]

Done.

 }