From d2bed2ba0434ba5259130581f2bd565640e5f53b Mon Sep 17 00:00:00 2001 From: Jonathan Almeida Date: Wed, 10 Aug 2022 22:19:12 -0400 Subject: [PATCH] [fenix] Close https://github.com/mozilla-mobile/fenix/issues/26041: Remove un-needed Nimbus workaround This workaround was temporary and is not needed with the Nimbus groovy plugin updates in Android Components. --- .../app/src/main/java/org/mozilla/fenix/FenixApplication.kt | 9 --------- 1 file changed, 9 deletions(-) diff --git a/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/FenixApplication.kt b/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/FenixApplication.kt index c7d9b2f356e2..ec143043bf3b 100644 --- a/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/FenixApplication.kt +++ b/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/FenixApplication.kt @@ -434,15 +434,6 @@ open class FenixApplication : LocaleAwareApplication(), Provider { private fun setupNimbusObserver(nimbus: Observable) { nimbus.register(object : NimbusInterface.Observer { override fun onUpdatesApplied(updated: List) { - // To workaround a caching bug in Nimbus that appears when we try to query an - // experiment **before** `FxNimbus.initialize` is called we need to set the `api` - // value again so that we can still access the NimbusApi that is wrapped - // in `FxNimbus.initialize.getSdk`. - // - // We set this value here to minimize the race between setting the `api` value and - // the callers of FxNimbus. - // See: https://github.com/mozilla/application-services/issues/5075 - FxNimbus.api = components.analytics.experiments onNimbusStartupAndUpdate() } }) -- 2.11.4.GIT