From 12840967223a9b241d2375562a80876954703c91 Mon Sep 17 00:00:00 2001 From: Arturo Mejia Date: Mon, 22 Nov 2021 15:15:35 -0500 Subject: [PATCH] [fenix] For issue https://github.com/mozilla-mobile/fenix/issues/22537 crash: BookmarkFragment is attached to a context --- .../mozilla/fenix/library/bookmarks/BookmarkFragment.kt | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt index 320e5af78..c4a09deba 100644 --- a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt @@ -261,9 +261,12 @@ class BookmarkFragment : LibraryPageFragment(), UserInteractionHan } private suspend fun loadBookmarkNode(guid: String): BookmarkNode? = withContext(IO) { - requireContext().bookmarkStorage - .getTree(guid, false) - ?.let { desktopFolders.withOptionalDesktopFolders(it) } + // Only runs if the fragment is attached same as [runIfFragmentIsAttached] + context?.let { + requireContext().bookmarkStorage + .getTree(guid, false) + ?.let { desktopFolders.withOptionalDesktopFolders(it) } + } } private suspend fun refreshBookmarks() {