Android: Record UMA for custom resources on startup.

Record UMA once during startup only for the browser process.

Bug: 768497
Change-Id: I24fa3eebe812ee4dc423390535e7b52ab8c6fcd1
Reviewed-on: https://chromium-review.googlesource.com/699697
Commit-Queue: Eric Stevenson <estevenson@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506595}

build/android/buildhooks/java/org/chromium/build/BuildHooksAndroid.java

@@ -62,4 +62,13 @@ public static void initCustomResources(Context context) {
     }
 
     protected abstract void initCustomResourcesImpl(Context context);
+
+    /**
+     * Record custom resources related UMA. Requires native library to be loaded.
+     */
+    public static void maybeRecordResourceMetrics() {
+        sInstance.maybeRecordResourceMetricsImpl();
+    }
+
+    protected abstract void maybeRecordResourceMetricsImpl();
 }

build/android/buildhooks/java/org/chromium/build/BuildHooksAndroidImpl.java

@@ -35,4 +35,6 @@ protected boolean isEnabledImpl() {
     }
 
     protected void initCustomResourcesImpl(Context context) {}
+
+    protected void maybeRecordResourceMetricsImpl() {}
 }

chrome/android/java/src/org/chromium/chrome/browser/init/ProcessInitializationHandler.java

@@ -29,6 +29,7 @@
 import org.chromium.base.TraceEvent;
 import org.chromium.base.annotations.SuppressFBWarnings;
 import org.chromium.base.metrics.RecordHistogram;
+import org.chromium.build.BuildHooksAndroid;
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.AfterStartupTaskUtils;
 import org.chromium.chrome.browser.AppHooks;
@@ -410,6 +411,9 @@ public void run() {
                 logEGLShaderCacheSizeHistogram();
             }
         });
+
+        deferredStartupHandler.addDeferredTask(
+                () -> { BuildHooksAndroid.maybeRecordResourceMetrics(); });
     }
 
     private void initChannelsAsync() {

tools/metrics/histograms/enums.xml

@@ -860,6 +860,12 @@ uploading your change for review.  These are checked by presubmit scripts.
   <int value="3" label="LowMemory"/>
 </enum>
 
+<enum name="AndroidResourceExtractionStatus">
+  <int value="0" label="No extraction performed"/>
+  <int value="1" label="Extraction success"/>
+  <int value="2" label="Extraction failure"/>
+</enum>
+
 <enum name="AndroidRestoreResult">
   <int value="0" label="No restore attempted"/>
   <int value="1" label="Restore successful"/>

tools/metrics/histograms/histograms.xml

@@ -716,6 +716,16 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Android.CompressedResources.ExtractionStatus"
+    enum="AndroidResourceExtractionStatus">
+  <owner>estevenson@chromium.org</owner>
+  <owner>agrieve@chromium.org</owner>
+  <summary>
+    Describes the result of Android resource extraction, and is recorded once
+    per browser start as part of deferred startup (not at time of extraction).
+  </summary>
+</histogram>
+
 <histogram name="Android.CompressedResources.ExtractionTime" units="ms">
   <owner>estevenson@chromium.org</owner>
   <owner>agrieve@chromium.org</owner>