From 569b7670d364d92a3501540aa376c062ed27d5b4 Mon Sep 17 00:00:00 2001 From: Severin Rudie Date: Tue, 7 Jan 2020 10:47:33 -0800 Subject: [PATCH] For 6832: update implementation/comment to align w existing AC code --- app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt index 7c89046e9..5d30f6b6d 100644 --- a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt @@ -230,7 +230,9 @@ class HomeFragment : Fragment() { setupHomeMenu() viewLifecycleOwner.lifecycleScope.launch(IO) { - // This should be unnecessary, but we've seen crashes around it. See #6832 + // This is necessary due to a bug in viewLifecycleOwner. See: + // https://github.com/mozilla-mobile/android-components/blob/master/components/lib/state/src/main/java/mozilla/components/lib/state/ext/Fragment.kt#L32-L56 + // TODO remove when viewLifecycleOwner is fixed val context = context ?: return@launch val iconSize = resources.getDimensionPixelSize(R.dimen.preference_icon_drawable_size)