Pressing back after dismissing keyboard in find in page close find in page view (#797)

* Fixed backButton behavior when findInPage is active

* Added test for backButton behavior when findInPage is active

Author: AndrewR8

app/src/androidTest/java/com/duckduckgo/app/browser/BrowserTabViewModelTest.kt

@@ -872,6 +872,19 @@ class BrowserTabViewModelTest {
         assertTrue(browserViewState().canGoForward)
     }
 
+    @Test
+    fun whenFindInPageShowingByPressingBackOnBrowserThenViewStateUpdatedInvisibleAndDoesNotGoToPreviousPage() {
+        setupNavigation(isBrowsing = true, canGoBack = true)
+        testee.onFindInPageSelected()
+        testee.onUserPressedBack()
+
+        assertFalse(findInPageViewState().visible)
+        assertCommandIssued<Command.DismissFindInPage>()
+
+        val issuedCommand = commandCaptor.allValues.find { it is Command.NavigateBack }
+        assertNull(issuedCommand)
+    }
+
     @Test
     fun whenHomeShowingByPressingBackOnInvalidatedBrowserThenForwardButtonInactive() {
         setupNavigation(isBrowsing = true)

app/src/main/java/com/duckduckgo/app/browser/BrowserTabViewModel.kt

@@ -415,6 +415,11 @@ class BrowserTabViewModel(
     fun onUserPressedBack(): Boolean {
         val navigation = webNavigationState ?: return false
 
+        if (currentFindInPageViewState().visible) {
+            dismissFindInView()
+            return true
+        }
+
         if (!currentBrowserViewState().browserShowing) {
             return false
         }