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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/ClearBrowsingDataPreferences.java

Issue 2455803002: Enforce min expected value for CustomCountHistograms on android (Closed)
Patch Set: Created 4 years, 2 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/preferences/privacy/ClearBrowsingDataPreferences.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/ClearBrowsingDataPreferences.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/ClearBrowsingDataPreferences.java
index abf6df5d37c74595ce3b40dd6a250e293c3b5f71..a8e4f3ce9c5c77feb162e5e204a69f4be1e7c295 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/ClearBrowsingDataPreferences.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/privacy/ClearBrowsingDataPreferences.java
@@ -618,10 +618,10 @@ public class ClearBrowsingDataPreferences extends PreferenceFragment
// mMaxImportantSites is a constant on the C++ side.
RecordHistogram.recordCustomCountHistogram(
"History.ClearBrowsingData.ImportantDeselectedNum",
- deselectedDomains.length, 0, mMaxImportantSites + 1,
+ deselectedDomains.length, 1, mMaxImportantSites + 1,
mMaxImportantSites + 1);
RecordHistogram.recordCustomCountHistogram(
- "History.ClearBrowsingData.ImportantIgnoredNum", ignoredDomains.length, 0,
+ "History.ClearBrowsingData.ImportantIgnoredNum", ignoredDomains.length, 1,
mMaxImportantSites + 1, mMaxImportantSites + 1);
// We put our max at 20 instead of 100 to reduce the number of empty buckets (as
// our maximum denominator is 5).

Powered by Google App Engine
This is Rietveld 408576698