From 1b2c0bd7999f68c0e981338a3fd4b61073ad10a1 Mon Sep 17 00:00:00 2001 From: Severin Rudie Date: Mon, 6 Jan 2020 14:22:32 -0800 Subject: [PATCH] [fenix] For https://github.com/mozilla-mobile/fenix/issues/6832: fix unattached context crash in HomeFragment --- app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt | 5 ++++- 1 file changed, 4 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 6982632508..7c89046e9d 100644 --- 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)