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

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

Issue 2910313002: [Android WebAPK] Add UMA histogram to record error code when install fails pt 2/3 (Closed)
Patch Set: Merge branch 'master' into more_uma Created 3 years, 6 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
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/webapps/GooglePlayWebApkInstallDelegate.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/metrics/WebApkUma.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/metrics/WebApkUma.java b/chrome/android/java/src/org/chromium/chrome/browser/metrics/WebApkUma.java
index 100e403d9307ed8e4872fe7d15614bbc18ad3feb..75850a2dea08197f32fb3ff39028d4650d668da5 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/metrics/WebApkUma.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/metrics/WebApkUma.java
@@ -95,6 +95,14 @@ public class WebApkUma {
"WebApk.Install.GooglePlayInstallResult", result, GOOGLE_PLAY_INSTALL_RESULT_MAX);
}
+ /** Records the error code if installing a WebAPK via Google Play fails. */
+ public static void recordGooglePlayInstallErrorCode(int errorCode) {
+ // Don't use an enumerated histogram as there are > 30 potential error codes. In practice,
+ // a given client will always get the same error code.
+ RecordHistogram.recordSparseSlowlyHistogram(
+ "WebApk.Install.GooglePlayErrorCode", Math.min(errorCode, 1000));
+ }
+
/**
* Records whether updating a WebAPK from Google Play succeeded. If not, records the reason
* that the update failed.
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/webapps/GooglePlayWebApkInstallDelegate.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698