diff --git a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkController.kt b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkController.kt index 385849a24a62..629db7611a9e 100644 --- a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkController.kt +++ b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkController.kt @@ -77,7 +77,7 @@ class DefaultBookmarkController( val flags = EngineSession.LoadUrlFlags.select(EngineSession.LoadUrlFlags.ALLOW_JAVASCRIPT_URL) openInNewTabAndShow( item.url!!, - false, + true, BrowserDirection.FromBookmarks, activity.browsingModeManager.mode, flags diff --git a/app/src/test/java/org/mozilla/fenix/library/bookmarks/BookmarkControllerTest.kt b/app/src/test/java/org/mozilla/fenix/library/bookmarks/BookmarkControllerTest.kt index 226ac4e7408c..b72a931823f6 100644 --- a/app/src/test/java/org/mozilla/fenix/library/bookmarks/BookmarkControllerTest.kt +++ b/app/src/test/java/org/mozilla/fenix/library/bookmarks/BookmarkControllerTest.kt @@ -110,7 +110,7 @@ class BookmarkControllerTest { } @Test - fun `handleBookmarkTapped should load the bookmark in the current tab`() { + fun `handleBookmarkTapped should load the bookmark in a new tab`() { var invokePendingDeletionInvoked = false val flags = EngineSession.LoadUrlFlags.select(EngineSession.LoadUrlFlags.ALLOW_JAVASCRIPT_URL) @@ -124,7 +124,7 @@ class BookmarkControllerTest { verify { homeActivity.openToBrowserAndLoad( item.url!!, - false, + true, BrowserDirection.FromBookmarks, flags = flags ) @@ -290,7 +290,7 @@ class BookmarkControllerTest { verify { homeActivity.openToBrowserAndLoad( item.url!!, - false, + true, BrowserDirection.FromBookmarks, flags = flags )