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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/metrics/UmaSessionStats.java

Issue 1048803003: Revert of New UMA settings fragment for Chrome on Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: merge Created 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/metrics/UmaSessionStats.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/metrics/UmaSessionStats.java b/chrome/android/java/src/org/chromium/chrome/browser/metrics/UmaSessionStats.java
index 6fed71652e2dd2ad3a303e5d8c1a1f55d13b5bea..e3de19bafbc1a31750bf2c62603582c5c2c929b4 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/metrics/UmaSessionStats.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/metrics/UmaSessionStats.java
@@ -151,8 +151,8 @@ public class UmaSessionStats implements NetworkChangeNotifier.ConnectionTypeObse
* Updates the state of the MetricsService to account for the user's preferences.
*/
public void updateMetricsServiceState() {
- boolean mayRecordStats =
- PrivacyPreferencesManager.getInstance(mContext).isUsageAndCrashReportingEnabled();
+ boolean mayRecordStats = !PrivacyPreferencesManager.getInstance(mContext)
+ .isNeverUploadCrashDump();
boolean mayUploadStats = mReportingPermissionManager.isUploadPermitted();
// Re-start the MetricsService with the given parameters.

Powered by Google App Engine
This is Rietveld 408576698