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

Side by Side Diff: components/minidump_uploader/android/java/src/org/chromium/components/minidump_uploader/util/CrashReportingPermissionManager.java

Issue 2756463004: [Cleanup] Remove the ability to disable Crash Reporting from the command line. (Closed)
Patch Set: Created 3 years, 9 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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.components.minidump_uploader.util; 5 package org.chromium.components.minidump_uploader.util;
6 6
7 /** 7 /**
8 * Interface for crash reporting permissions. 8 * Interface for crash reporting permissions.
9 */ 9 */
10 public interface CrashReportingPermissionManager { 10 public interface CrashReportingPermissionManager {
(...skipping 15 matching lines...) Expand all
26 // TODO(isherman): Remove this function. It's only used for an assertion, an d our JobService 26 // TODO(isherman): Remove this function. It's only used for an assertion, an d our JobService
27 // implementations simply hardcode their return value to true. 27 // implementations simply hardcode their return value to true.
28 /** 28 /**
29 * Checks whether uploading of usage metrics is currently permitted. This is a combination of 29 * Checks whether uploading of usage metrics is currently permitted. This is a combination of
30 * the below checks, plus networking restrictions. 30 * the below checks, plus networking restrictions.
31 * 31 *
32 * @return whether uploading usage metrics is currently permitted. 32 * @return whether uploading usage metrics is currently permitted.
33 */ 33 */
34 public boolean isMetricsUploadPermitted(); 34 public boolean isMetricsUploadPermitted();
35 35
36 // TODO(isherman): Remove this function. It was previously used to enable a specific type of
37 // test, but it should not be necessary anymore. (Note: There are currently some clients that do
38 // partly depend on it, so it is not necessarily safe to remove without cont acting those
39 // clients.)
40 /**
41 * Checks whether uploading of crash dumps is permitted, based on the corres ponding command line
42 * flag only.
43 *
44 * @return whether uploading of crash dumps is enabled or disabled by a comm and line flag.
45 */
46 public boolean isCrashUploadDisabledByCommandLine();
47
48 /** 36 /**
49 * Checks whether uploading of usage metrics and crash dumps is currently pe rmitted, based on 37 * Checks whether uploading of usage metrics and crash dumps is currently pe rmitted, based on
50 * user consent only. 38 * user consent only.
51 * 39 *
52 * @return whether the user has consented to reporting usage metrics and cra sh dumps. 40 * @return whether the user has consented to reporting usage metrics and cra sh dumps.
53 */ 41 */
54 public boolean isUsageAndCrashReportingPermittedByUser(); 42 public boolean isUsageAndCrashReportingPermittedByUser();
55 43
56 /** 44 /**
57 * Checks whether to ignore all consent and upload limitations for usage met rics and crash 45 * Checks whether to ignore all consent and upload limitations for usage met rics and crash
58 * reporting. Used by test devices to avoid a UI dependency. 46 * reporting. Used by test devices to avoid a UI dependency.
59 * 47 *
60 * @return whether crash dumps should be uploaded if at all possible. 48 * @return whether crash dumps should be uploaded if at all possible.
61 */ 49 */
62 public boolean isUploadEnabledForTests(); 50 public boolean isUploadEnabledForTests();
63 } 51 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698