Chromium Code Reviews| 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 2ebed96d34158787b336c2510d21812985576498..fb98c91138a7c7cab6ab93295ef88e714420b4c1 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 |
| @@ -153,7 +153,9 @@ public class UmaSessionStats implements NetworkChangeNotifier.ConnectionTypeObse |
| } |
| /** |
| - * Updates the state of the MetricsService to account for the user's preferences. |
| + * Updates the state of the MetricsService to account for the user's preferences. Android prefs |
| + * and Chrome Local State prefs for metrics reporting should be in sync before calling this |
| + * function. |
| */ |
| public void updateMetricsServiceState() { |
| boolean mayRecordStats = !PrivacyPreferencesManager.getInstance() |
| @@ -192,6 +194,9 @@ public class UmaSessionStats implements NetworkChangeNotifier.ConnectionTypeObse |
| prefManager.setUsageAndCrashReporting(prefBridge.isMetricsReportingEnabled()); |
| } |
| + // Update the metrics sampling state. |
|
Alexei Svitkine (slow)
2016/08/18 06:16:39
Expand comment to mention why this is being done -
jwd
2016/08/23 18:40:03
Done.
|
| + prefManager.setClientInMetricsSample(UmaUtils.isClientInSample()); |
| + |
| // Make sure preferences are in sync. |
| prefManager.syncUsageAndCrashReportingPrefs(); |
| } |