Index: chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkInstaller.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkInstaller.java b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkInstaller.java |
index 5523f9283e1794f30c787d7516d0d98bb8c48d8f..e981b0b474f57ebd9173543776a1352682d5176a 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkInstaller.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkInstaller.java |
@@ -20,6 +20,7 @@ import org.chromium.base.annotations.CalledByNative; |
import org.chromium.chrome.browser.AppHooks; |
import org.chromium.chrome.browser.ShortcutHelper; |
import org.chromium.chrome.browser.banners.InstallerDelegate; |
+import org.chromium.chrome.browser.metrics.WebApkUma; |
import org.chromium.chrome.browser.util.IntentUtils; |
import java.io.File; |
@@ -109,6 +110,8 @@ public class WebApkInstaller { |
String packageName, int version, String title, String token, String url) { |
if (mGooglePlayWebApkInstallDelegate == null) { |
notify(WebApkInstallResult.FAILURE); |
+ WebApkUma.recordGooglePlayIntallResult( |
dominickn
2017/03/22 00:15:22
Should this conditional be moved after the isWebAp
Xi Han
2017/03/22 17:32:36
It makes sense to me. Yes, we shouldn't records an
|
+ WebApkUma.GOOGLE_PLAY_INSTALL_FAILED_NO_DELEGATE); |
return; |
} |