Closes #1835 - Do not crash because of snackbar

6 years ago · 4b59fc8939
parent d1fd5ec879
commit 4b59fc8939
1 changed files with 17 additions and 15 deletions
--- a/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt
+++ b/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt
@ -358,8 +358,9 @@ class BrowserFragment : Fragment(), BackHandler, CoroutineScope {
                                .addItem(BookmarkRoot.Mobile.id, session.url, session.title, null)
                            launch(Main) {
                                requireComponents.analytics.metrics.track(Event.AddBookmark)
+                                view?.let {
                                    FenixSnackbar.make(
-                                    view!!,
+                                        it,
                                        Snackbar.LENGTH_LONG
                                    )
                                        .setAction(getString(R.string.edit_bookmark_snackbar_action)) {
@ -376,6 +377,7 @@ class BrowserFragment : Fragment(), BackHandler, CoroutineScope {
                                }
                            }
                        }
+                    }
                    is QuickActionAction.ReadPressed -> {
                        requireComponents.analytics.metrics.track(Event.QuickActionSheetReadTapped)
                        ItsNotBrokenSnack(context!!).showSnackbar(issueNumber = "908")