Enabling sampling of UMA and crash reports on Android.

chrome/android/java/src/org/chromium/chrome/browser/crash/MinidumpUploadCallable.java

Index: chrome/android/java/src/org/chromium/chrome/browser/crash/MinidumpUploadCallable.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/crash/MinidumpUploadCallable.java b/chrome/android/java/src/org/chromium/chrome/browser/crash/MinidumpUploadCallable.java
index d21a02a6553a98b47f7835c725a31586d9081a56..2fe7bafe82beaf67fac5b8484338d63d706b32c6 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/crash/MinidumpUploadCallable.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/crash/MinidumpUploadCallable.java
@@ -107,6 +107,11 @@ public class MinidumpUploadCallable implements Callable<Integer> {
                 Log.i(TAG, "Minidump cannot currently be uploaded due to constraints.");
                 return UPLOAD_FAILURE;
             }
+
+            if (!mPermManager.isClientInMetricsSample()) {
+                Log.i(TAG, "Minidump cannot currently be uploaded due to sampling.");
+                return UPLOAD_FAILURE;

[Alexei Svitkine (slow), 2016/08/18 06:16:39]

Can you add a new enum for this case. I think it's confusing to return
UPLOAD_FAILURE here.

You can make the caller(s) of this handle it the same way as UPLOAD_FAILURE, if
that's what makes the most sense, but I think having the distinction makes
sense.

[jwd, 2016/08/23 18:40:03]

Done.

+            }
         }
 
         HttpURLConnection connection =