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 629db7611a9e..385849a24a62 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!!, - true, + false, 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 b72a931823f6..226ac4e7408c 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 a new tab`() { + fun `handleBookmarkTapped should load the bookmark in the current 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!!, - true, + false, BrowserDirection.FromBookmarks, flags = flags ) @@ -290,7 +290,7 @@ class BookmarkControllerTest { verify { homeActivity.openToBrowserAndLoad( item.url!!, - true, + false, BrowserDirection.FromBookmarks, flags = flags )