[Offline pages] Creating BackgroundJobScheduler, which uses JobScheduler

chrome/android/java/src/org/chromium/chrome/browser/offlinepages/DeviceConditions.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;
 
 /** Device network and power conditions. */
 public class DeviceConditions {
     private final boolean mPowerConnected;
     private final int mBatteryPercentage;
     private final int mNetConnectionType;
 
     /**
      * Creates set of device network and power conditions.
      * @param powerConnected whether device is connected to power
      * @param batteryPercentage percentage (0-100) of remaining battery power
      * @param connectionType the org.chromium.net.ConnectionType value for the network connection
      */
     public DeviceConditions(boolean powerConnected, int batteryPercentage, int netConnectionType) {
         mPowerConnected = powerConnected;
         mBatteryPercentage = batteryPercentage;
         mNetConnectionType = netConnectionType;
     }
 
+    /** @return Whether power is connected. */
     public boolean isPowerConnected() {
         return mPowerConnected;
     }
 
+    /** @return Battery percentage. */
     public int getBatteryPercentage() {
         return mBatteryPercentage;
     }
 
-    /**
-     * Returns the Chromium enum value for the network connection type. Connection type values are
-     * defined in org.chromium.net.ConnectionType.
-     */
+    /** @return Network connection type as defined by org.chroium.net.ConnectionType. */

[Pete Williamson, 2017/02/10 02:10:43]

Comment might be slightly misleading.  We actually return an int with the same
values as org.chromium.net.ConnectionType, but not an instance of
ConnectionType.

[fgorski, 2017/02/10 19:26:00]

Updated.

     public int getNetConnectionType() {
         return mNetConnectionType;
     }
 }