Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageUtils.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageUtils.java b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageUtils.java |
| index 34aba7a8fb834121a67ab1593fe3113f09724706..bb580f22d5ccd2f0fe52d2a9996052635f728e58 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageUtils.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageUtils.java |
| @@ -9,7 +9,6 @@ import android.content.Intent; |
| import android.content.IntentFilter; |
| import android.os.BatteryManager; |
| import android.os.Environment; |
| -import android.os.SystemClock; |
| import org.chromium.base.Log; |
| import org.chromium.base.metrics.RecordHistogram; |
| @@ -206,7 +205,7 @@ public class OfflinePageUtils { |
| * Records UMA data when the Offline Pages Background Load service awakens. |
| * @param context android context |
| */ |
| - public static void recordWakeupUMA(Context context, long millisSinceBootTask) { |
| + public static void recordWakeupUMA(Context context, long millisTaskStart) { |
|
dewittj
2016/06/03 22:02:46
taskScheduledTimeMillis might be a better name
dougarnett
2016/06/03 22:45:50
+1
Pete Williamson
2016/06/03 23:42:11
Done.
|
| IntentFilter filter = new IntentFilter(Intent.ACTION_BATTERY_CHANGED); |
| // Note this is a sticky intent, so we aren't really registering a receiver, just getting |
| // the sticky intent. That means that we don't need to unregister the filter later. |
| @@ -228,8 +227,8 @@ public class OfflinePageUtils { |
| connectionType, ConnectionType.CONNECTION_LAST + 1); |
| // Collect UMA on the time since the request started. |
| - long milliSinceBootNow = SystemClock.elapsedRealtime(); |
| - long delayInMilliseconds = milliSinceBootNow - millisSinceBootTask; |
| + long millisNow = System.currentTimeMillis(); |
|
dougarnett
2016/06/03 22:45:50
nowMillis
Pete Williamson
2016/06/03 23:42:11
Done.
|
| + long delayInMilliseconds = millisNow - millisTaskStart; |
| if (delayInMilliseconds <= 0) { |
| return; |
| } |