For #6832: fix unattached context crash in HomeFragment

mozilla-mobile · Jan 8, 2020 · 65f5ff6 · 65f5ff6
1 parent b115a00
commit 65f5ff6
Showing 1 changed file with 4 additions and 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
@@ -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)