Defines TriggerConditions and plumbs up to BackgroundScheduler

chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerBridge.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 org.chromium.base.Callback;
 import org.chromium.base.ContextUtils;
 import org.chromium.base.annotations.CalledByNative;
 import org.chromium.base.annotations.JNINamespace;
@@ skipping 14 matching lines @@
     // TODO(dougarnett): consider adding policy check api to let caller
     //     separately determine if not allowed by policy.
     public static boolean startProcessing(
             DeviceConditions deviceConditions, Callback<Boolean> callback) {
         return nativeStartProcessing(deviceConditions.isPowerConnected(),
                 deviceConditions.getBatteryPercentage(), deviceConditions.getNetConnectionType(),
                 callback);
     }
 
     @CalledByNative
-    private static void schedule() {
-        BackgroundScheduler.schedule(ContextUtils.getApplicationContext());
+    private static void schedule(TriggerConditions triggerConditions) {
+        BackgroundScheduler.schedule(ContextUtils.getApplicationContext(), triggerConditions);
     }
 
     @CalledByNative
     private static void unschedule() {
         BackgroundScheduler.unschedule(ContextUtils.getApplicationContext());
     }
 
+    @CalledByNative

[Pete Williamson, 2016/06/20 23:14:03]

We might document *why* we have a createTriggerContidions function instead of
just calling the constructor (so we can create the java object from native
code).

[dougarnett, 2016/06/22 19:54:18]

Done.

+    private static TriggerConditions createTriggerConditions(boolean requirePowerConnected,
+            int minimumBatteryPercentage, boolean requireUnmeteredNetwork) {
+        return new TriggerConditions(
+                requirePowerConnected, minimumBatteryPercentage, requireUnmeteredNetwork);
+    }
+
     private static native boolean nativeStartProcessing(boolean powerConnected,
             int batteryPercentage, int netConnectionType, Callback<Boolean> callback);
 }