Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Commit

Permalink
Close #26041: Remove un-needed Nimbus workaround
Browse files Browse the repository at this point in the history
This workaround was temporary and is not needed with the Nimbus groovy
plugin updates in Android Components.
  • Loading branch information
jonalmeida committed Aug 11, 2022
1 parent 3fdff35 commit 0a56ead
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions app/src/main/java/org/mozilla/fenix/FenixApplication.kt
Original file line number Diff line number Diff line change
Expand Up @@ -425,15 +425,6 @@ open class FenixApplication : LocaleAwareApplication(), Provider {
private fun setupNimbusObserver(nimbus: Observable<NimbusInterface.Observer>) {
nimbus.register(object : NimbusInterface.Observer {
override fun onUpdatesApplied(updated: List<EnrolledExperiment>) {
// 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()
}
})
Expand Down

0 comments on commit 0a56ead

Please sign in to comment.