Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(137)

Side by Side Diff: chrome/android/java/src/org/chromium/chrome/browser/crash/MinidumpUploadService.java

Issue 2751333004: [Crash Reporting] Only upload Chrome crash reports over unmetered networks. (Closed)
Patch Set: Rebase Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser.crash; 5 package org.chromium.chrome.browser.crash;
6 6
7 import android.annotation.SuppressLint; 7 import android.annotation.SuppressLint;
8 import android.app.IntentService; 8 import android.app.IntentService;
9 import android.app.job.JobInfo; 9 import android.app.job.JobInfo;
10 import android.content.ComponentName; 10 import android.content.ComponentName;
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 PersistableBundle permissions = new PersistableBundle(); 98 PersistableBundle permissions = new PersistableBundle();
99 permissions.putBoolean(ChromeMinidumpUploaderDelegate.IS_CLIENT_IN_METRI CS_SAMPLE, 99 permissions.putBoolean(ChromeMinidumpUploaderDelegate.IS_CLIENT_IN_METRI CS_SAMPLE,
100 permissionManager.isClientInMetricsSample()); 100 permissionManager.isClientInMetricsSample());
101 permissions.putBoolean(ChromeMinidumpUploaderDelegate.IS_UPLOAD_ENABLED_ FOR_TESTS, 101 permissions.putBoolean(ChromeMinidumpUploaderDelegate.IS_UPLOAD_ENABLED_ FOR_TESTS,
102 permissionManager.isUploadEnabledForTests()); 102 permissionManager.isUploadEnabledForTests());
103 103
104 JobInfo.Builder builder = 104 JobInfo.Builder builder =
105 new JobInfo 105 new JobInfo
106 .Builder(TaskIds.CHROME_MINIDUMP_UPLOADING_JOB_ID, 106 .Builder(TaskIds.CHROME_MINIDUMP_UPLOADING_JOB_ID,
107 new ComponentName(context, ChromeMinidumpUploadJ obService.class)) 107 new ComponentName(context, ChromeMinidumpUploadJ obService.class))
108 .setRequiredNetworkType(JobInfo.NETWORK_TYPE_ANY)
109 .setExtras(permissions); 108 .setExtras(permissions);
110 MinidumpUploadJobService.scheduleUpload(context, builder); 109 MinidumpUploadJobService.scheduleUpload(context, builder);
111 } 110 }
112 111
113 /** 112 /**
114 * Stores the successes and failures from uploading crash to UMA, 113 * Stores the successes and failures from uploading crash to UMA,
115 */ 114 */
116 public static void storeBreakpadUploadStatsInUma(ChromePreferenceManager pre f) { 115 public static void storeBreakpadUploadStatsInUma(ChromePreferenceManager pre f) {
117 for (String type : TYPES) { 116 for (String type : TYPES) {
118 for (int success = pref.getCrashSuccessUploadCount(type); success > 0; success--) { 117 for (int success = pref.getCrashSuccessUploadCount(type); success > 0; success--) {
(...skipping 242 matching lines...) Expand 10 before | Expand all | Expand 10 after
361 return; 360 return;
362 } 361 }
363 362
364 if (shouldUseJobSchedulerForUploads()) { 363 if (shouldUseJobSchedulerForUploads()) {
365 scheduleUploadJob(context); 364 scheduleUploadJob(context);
366 } else { 365 } else {
367 tryUploadCrashDump(context, renamedMinidumpFile); 366 tryUploadCrashDump(context, renamedMinidumpFile);
368 } 367 }
369 } 368 }
370 } 369 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698