Currently, I'm only checking the MOZ_UPDATER build define, which should exclude Beta and Release, but I should additionally be checking the installation source as well, so the notification channel only shows up on Nightly builds *not* installed from the Play Store.

That way, we only add that notification channel if the updater really is active, i.e. not if we've been installed from the Play Store. Unfortunately, using the UpdateServiceHelper requires a Context, so adding/re- moving the UPDATER_CHANNEL_TAG cannot be done during variable initialisation.

Comment on attachment 9006163 [details] Bug 1488174 - Use UpdateServiceHelper to decide on whether to add Updater notification channel. r?jchen Jim Chen [:jchen] [:darchons] has approved the revision.

Pushed by mozilla@buttercookie.de: https://hg.mozilla.org/integration/autoland/rev/b9e56df24edc Use UpdateServiceHelper to decide on whether to add Updater notification channel. r=jchen

Backout by ccoroiu@mozilla.com: https://hg.mozilla.org/mozilla-central/rev/54cd02c97ed4 Backed out changeset b9e56df24edc for fennec crashing at startup a=backout

This got backed out request from QA and the SoftVision Android developers: https://hg.mozilla.org/mozilla-central/rev/54cd02c97ed4 Tracked in bug 1489168.

Hmm, it seems the ArrayList returned from Arrays.asList isn't actually extendable.

Beta and Release have MOZ_UPDATER as false, so no need to uplift.

Pushed by mozilla@buttercookie.de: https://hg.mozilla.org/integration/autoland/rev/e74bd1eae0db Use UpdateServiceHelper to decide on whether to add Updater notification channel. r=jchen