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

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

Issue 1909483003: [MediaNotification] Add UMA to record user clicking media notifications (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: renaming Created 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchMetrics.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchMetrics.java b/chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchMetrics.java
index 93576f161af44d0c5f51efe839f430b64d2cc9b4..87e457bebf7903feaae2711693e3810b91e5d987 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchMetrics.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchMetrics.java
@@ -84,6 +84,29 @@ public class LaunchMetrics {
}
}
+ /** Caches a set of enumerated histogram samples. */
+ public static class EnumeratedHistogramSample extends CachedHistogram {
+ private final List<Integer> mSamples = new ArrayList<Integer>();
+ private final int mMaxValue;
+
+ public EnumeratedHistogramSample(String histogramName, int maxValue) {
+ super(histogramName);
+ mMaxValue = maxValue;
+ }
+
+ public void record(int sample) {
+ mSamples.add(sample);
+ }
+
+ @Override
+ protected void commitAndClear() {
+ for (Integer sample : mSamples) {
+ RecordHistogram.recordEnumeratedHistogram(mHistogramName, sample, mMaxValue);
+ }
+ mSamples.clear();
+ }
+ }
+
// Each list item is a pair of the url and where it was added from e.g. from the add to
// homescreen menu item, an app banner, or unknown. The mapping of int source values to
// their string names is found in the C++ ShortcutInfo struct.

Powered by Google App Engine
This is Rietveld 408576698