Index: android_webview/java/src/org/chromium/android_webview/crash/AwMinidumpUploaderDelegate.java |
diff --git a/android_webview/java/src/org/chromium/android_webview/crash/AwMinidumpUploaderDelegate.java b/android_webview/java/src/org/chromium/android_webview/crash/AwMinidumpUploaderDelegate.java |
index 1fe38fbd95e525d2b77911e66251e7ed0e074536..9be1b7715df5ec5c8b5fe712a957efade1803d03 100644 |
--- a/android_webview/java/src/org/chromium/android_webview/crash/AwMinidumpUploaderDelegate.java |
+++ b/android_webview/java/src/org/chromium/android_webview/crash/AwMinidumpUploaderDelegate.java |
@@ -6,7 +6,6 @@ package org.chromium.android_webview.crash; |
import android.content.Context; |
import android.net.ConnectivityManager; |
-import android.net.NetworkInfo; |
import android.webkit.ValueCallback; |
import org.chromium.android_webview.PlatformServiceBridge; |
@@ -16,6 +15,7 @@ import org.chromium.base.ThreadUtils; |
import org.chromium.base.VisibleForTesting; |
import org.chromium.components.minidump_uploader.MinidumpUploaderDelegate; |
import org.chromium.components.minidump_uploader.util.CrashReportingPermissionManager; |
+import org.chromium.components.minidump_uploader.util.NetworkPermissionUtil; |
import java.io.File; |
@@ -54,14 +54,10 @@ public class AwMinidumpUploaderDelegate implements MinidumpUploaderDelegate { |
} |
@Override |
public boolean isNetworkAvailableForCrashUploads() { |
- // JobScheduler will call onStopJob causing our upload to be interrupted when our |
- // network requirements no longer hold. |
- // TODO(isherman): This code should really be shared with Chrome. Chrome currently |
- // checks only whether the network is WiFi (or ethernet) vs. cellular. Most likely, |
- // Chrome should instead check whether the network is metered, as is done here. |
- NetworkInfo networkInfo = mConnectivityManager.getActiveNetworkInfo(); |
- if (networkInfo == null || !networkInfo.isConnected()) return false; |
- return !mConnectivityManager.isActiveNetworkMetered(); |
+ // Note that this is the same critierion that the JobScheduler uses to schedule the |
+ // job. JobScheduler will call onStopJob causing our upload to be interrupted when |
+ // our network requirements no longer hold. |
+ return NetworkPermissionUtil.isNetworkUnmetered(mConnectivityManager); |
} |
@Override |
public boolean isUsageAndCrashReportingPermittedByUser() { |