🔍 Don't display the search engine until First Run completes

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

 // Copyright 2015 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.init;
 
 import android.annotation.SuppressLint;
 import android.annotation.TargetApi;
 import android.app.Activity;
 import android.content.Context;
@@ skipping 20 matching lines @@
 import org.chromium.base.VisibleForTesting;
 import org.chromium.base.library_loader.LibraryLoader;
 import org.chromium.base.library_loader.LoaderErrors;
 import org.chromium.base.library_loader.ProcessInitException;
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.ChromeApplication;
 import org.chromium.chrome.browser.WarmupManager;
 import org.chromium.chrome.browser.firstrun.FirstRunFlowSequencer;
 import org.chromium.chrome.browser.metrics.MemoryUma;
 import org.chromium.chrome.browser.profiles.Profile;
+import org.chromium.chrome.browser.searchwidget.SearchWidgetProvider;
 import org.chromium.chrome.browser.tabmodel.DocumentModeAssassin;
 import org.chromium.chrome.browser.upgrade.UpgradeActivity;
 import org.chromium.ui.base.ActivityWindowAndroid;
 import org.chromium.ui.base.DeviceFormFactor;
 import org.chromium.ui.base.WindowAndroid;
 
 import java.lang.reflect.Field;
 
 /**
  * An activity that talks with application and activity level delegates for async initialization.
@@ skipping 154 matching lines @@
                 });
         mMemoryUma = new MemoryUma();
         mNativeInitializationController.onNativeInitializationComplete();
     }
 
     /**
      * Actions that may be run at some point after startup. Place tasks that are not critical to the
      * startup path here.  This method will be called automatically and should not be called
      * directly by subclasses.
      */
-    protected void onDeferredStartup() { }
+    protected void onDeferredStartup() {
+        mHandler.post(new Runnable() {

[Ted C, 2017/04/22 20:17:06]

same question as the other init, do you want this to be per activity or just
once per process initialization?

If the latter, I think it is better to go in
ProcessInitializationHandler#handleDeferredStartupTasksInitialization
https://cs.chromium.org/chromium/src/chrome/android/java/src/org/chromium/chr...

[gone, 2017/04/23 21:00:34]

Moved it to when the FirstRunActivity marks itself compelte.  I'll need another
check once the the promo dialog is in to force a widget update there.

+            @Override
+            public void run() {
+                // Update the search engine name cached by the widget.
+                SearchWidgetProvider.updateCachedEngineName();
+            }
+        });
+    }
 
     @Override
     public void onStartupFailure() {
         ProcessInitException e =
                 new ProcessInitException(LoaderErrors.LOADER_ERROR_NATIVE_STARTUP_FAILED);
         ChromeApplication.reportStartupErrorAndExit(e);
     }
 
     /**
      * Extending classes should override {@link AsyncInitializationActivity#preInflationStartup()},
@@ skipping 430 matching lines @@
                             getDecorView().setVisibility(View.GONE);
                         }
                         getViewTreeObserver().removeOnPreDrawListener(mPreDrawListener);
                     }
                 });
                 return true;
             }
         };
     }
 }