Android: Make the DFM initialization call more explicit

Moved the implicit initialization of DFMs from a logger call
into ChromeApplication. This is a no-op, but should help in debugging
future startup issues with DFMs.

Bug: 1022654
Change-Id: Ia3de76f033058d01468333fed10db108cfc8c316
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2156607
Commit-Queue: Fred Mello <fredmello@chromium.org>
Reviewed-by: Samuel Huang <huangs@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#760951}

chrome/android/java/src/org/chromium/chrome/browser/ChromeApplication.java

@@ -119,9 +119,8 @@ public class ChromeApplication extends Application {
             ApplicationStatus.registerApplicationStateListener(
                     ChromeApplication::updateMemoryPressurePolling);
 
-            // Record via UMA all modules that have been requested and are currently installed. This
-            // will tell us the install penetration of each module over time.
-            ModuleUtil.recordModuleAvailability();
+            // Initializes the support for dynamic feature modules (browser only).
+            ModuleUtil.initApplication();
 
             // Set Chrome factory for mapping BackgroundTask classes to TaskIds.
             ChromeBackgroundTaskFactory.setAsDefault();

components/module_installer/android/java/src/org/chromium/components/module_installer/util/ModuleUtil.java

@@ -23,18 +23,6 @@ public class ModuleUtil {
         Timer.recordStartupTime();
     }
 
-    /**
-     * Records the start time in order to later report the install duration via UMA.
-     */
-    public static void recordModuleAvailability() {
-        if (!BundleUtils.isBundle()) return;
-
-        try (Timer timer = new Timer()) {
-            initApplication();
-            SplitAvailabilityLogger.logModuleAvailability();
-        }
-    }
-
     /**
      * Updates the CrashKey report containing modules currently present.
      */
@@ -55,6 +43,7 @@ public class ModuleUtil {
         try (Timer timer = new Timer()) {
             SplitCompatInitializer.initApplication();
             ActivityObserverUtil.registerDefaultObserver();
+            SplitAvailabilityLogger.logModuleAvailability();
         }
     }