Use HomeFragment viewLifecycleOwner for BrowserSessionsObserver

fennec/production
ekager 4 years ago committed by Emily Kager
parent 0ec6d266e8
commit 6f77ec5951

@ -159,17 +159,6 @@ class HomeFragment : Fragment() {
override fun onCreate(savedInstanceState: Bundle?) { override fun onCreate(savedInstanceState: Bundle?) {
super.onCreate(savedInstanceState) super.onCreate(savedInstanceState)
postponeEnterTransition() postponeEnterTransition()
val sessionObserver = BrowserSessionsObserver(
sessionManager,
requireComponents.core.store,
singleSessionObserver
) {
emitSessionChanges()
}
lifecycle.addObserver(sessionObserver)
if (!onboarding.userHasBeenOnboarded()) { if (!onboarding.userHasBeenOnboarded()) {
requireComponents.analytics.metrics.track(Event.OpenedAppFirstRun) requireComponents.analytics.metrics.track(Event.OpenedAppFirstRun)
} }
@ -183,6 +172,16 @@ class HomeFragment : Fragment() {
val view = inflater.inflate(R.layout.fragment_home, container, false) val view = inflater.inflate(R.layout.fragment_home, container, false)
val activity = activity as HomeActivity val activity = activity as HomeActivity
val sessionObserver = BrowserSessionsObserver(
sessionManager,
requireComponents.core.store,
singleSessionObserver
) {
emitSessionChanges()
}
viewLifecycleOwner.lifecycle.addObserver(sessionObserver)
currentMode = CurrentMode( currentMode = CurrentMode(
view.context, view.context,
onboarding, onboarding,

Loading…
Cancel
Save