[Android O] Initialize channels on first launch/upgrade

chrome/android/java/src/org/chromium/chrome/browser/upgrade/PackageReplacedBroadcastReceiver.java

Index: chrome/android/java/src/org/chromium/chrome/browser/upgrade/PackageReplacedBroadcastReceiver.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/upgrade/PackageReplacedBroadcastReceiver.java b/chrome/android/java/src/org/chromium/chrome/browser/upgrade/PackageReplacedBroadcastReceiver.java
index 3b8782df95d4f7265d87a6d5fcf2a0923e4ff121..7b3d6b5c92c597cb2d3be100d7f7ae554e2b2c7f 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/upgrade/PackageReplacedBroadcastReceiver.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/upgrade/PackageReplacedBroadcastReceiver.java
@@ -9,6 +9,8 @@ import android.content.Context;
 import android.content.Intent;
 import android.os.Build;
 
+import org.chromium.chrome.browser.notifications.ChannelsUpdater;
+
 /**
  * Triggered when Chrome's package is replaced (e.g. when it is upgraded).
  *
@@ -27,6 +29,9 @@ public final class PackageReplacedBroadcastReceiver extends BroadcastReceiver {
     @Override
     public void onReceive(Context context, Intent intent) {
         if (!Intent.ACTION_MY_PACKAGE_REPLACED.equals(intent.getAction())) return;
+        if (ChannelsUpdater.shouldUpdateChannels()) {

[nyquist, 2017/04/10 21:04:30]

Do we want ChannelsUpdate.updateChannelsIfNecessary() which could contain this
logic?

[awdf, 2017/04/11 14:19:22]

This isn't applicable now that I've added the async task within the check (with
goAsync which is broadcast-receiver specific).

+            ChannelsUpdater.updateChannels(context);

[nyquist, 2017/04/10 21:04:30]

I think this might end up writing to disk?
Do we want to goAsync() here to give ourselves some more time?
See https://codereview.chromium.org/2805533007/ as an example.

It'll ensure you won't get ANRs or StrictMode violations, etc. So I think it's
the right thing to do. See
https://developer.android.com/reference/android/content/BroadcastReceiver.htm...
for details.

[awdf, 2017/04/11 14:19:21]

Ah cool I didn't know about that one - sounds like exactly what we need here.
Done.

+        }
         if (Build.VERSION.SDK_INT > Build.VERSION_CODES.N) return;
         UpgradeIntentService.startMigrationIfNecessary(context);
     }