Add metrics to track Google Play install WebAPK failures.

chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkInstaller.java

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.webapps;
 
 import android.content.Context;
 import android.content.Intent;
 import android.content.pm.PackageManager;
 import android.net.Uri;
 import android.os.Build;
 import android.os.Looper;
 
 import org.chromium.base.ApplicationState;
 import org.chromium.base.ApplicationStatus;
 import org.chromium.base.Callback;
 import org.chromium.base.ContentUriUtils;
 import org.chromium.base.ContextUtils;
 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;
 
 /**
  * Java counterpart to webapk_installer.h
  * Contains functionality to install / update WebAPKs.
  * This Java object is created by and owned by the native WebApkInstaller.
  */
 public class WebApkInstaller {
@@ skipping 69 matching lines @@
      * @param version The version of WebAPK to install.
      * @param title The title of the WebAPK to display during installation.
      * @param token The token from WebAPK Server.
      * @param url The start URL of the WebAPK to install.
      */
     @CalledByNative
     private void installWebApkFromGooglePlayAsync(
             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 isWebApkInstalled check? I'm thinking
of cases where you've managed to get a WebAPK installed, but then you trigger
another install and your Play Services isn't available. Seems like a rare case
but we may as well account for it if there are no side effects.

[Xi Han, 2017/03/22 17:32:36]

It makes sense to me. Yes, we shouldn't records anything if the WebAPK has been
installed. Moved.

+                    WebApkUma.GOOGLE_PLAY_INSTALL_FAILED_NO_DELEGATE);
             return;
         }
 
         // Check whether the WebAPK package is already installed. The WebAPK may have been installed
         // by another Chrome version (e.g. Chrome Dev). We have to do this check because the Play
         // install API fails silently if the package is already installed.
         if (isWebApkInstalled(packageName)) {
             notify(WebApkInstallResult.SUCCESS);
             return;
         }
@@ skipping 123 matching lines @@
     }
 
     private boolean isWebApkInstalled(String packageName) {
         PackageManager packageManager = ContextUtils.getApplicationContext().getPackageManager();
         return InstallerDelegate.isInstalled(packageManager, packageName);
     }
 
     private native void nativeOnInstallFinished(
             long nativeWebApkInstaller, @WebApkInstallResult.WebApkInstallResultEnum int result);
 }