For #6832: fix unattached context crash in HomeFragment

4 years ago · 65f5ff6b36
parent b115a00589
commit 65f5ff6b36
1 changed files with 4 additions and 1 deletions
--- a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt
+++ b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt
@ -230,9 +230,12 @@ class HomeFragment : Fragment() {
        setupHomeMenu()

        viewLifecycleOwner.lifecycleScope.launch(IO) {
+            // This should be unnecessary, but we've seen crashes around it. See #6832
+            val context = context ?: return@launch
+
            val iconSize = resources.getDimensionPixelSize(R.dimen.preference_icon_drawable_size)

-            val searchEngine = requireComponents.search.provider.getDefaultEngine(requireContext())
+            val searchEngine = requireComponents.search.provider.getDefaultEngine(context)
            val searchIcon = BitmapDrawable(resources, searchEngine.icon)
            searchIcon.setBounds(0, 0, iconSize, iconSize)