Remove showOmnibar from navigationStateChanged to fix easter egg logos

This is a leftover from Josh's loading bar experiment. Ondrej needed to account for that when adding the new Bottom Omnibar here but as Josh has removed the experiment it is no longer needed. I tested and it makes no difference and now acts the same way as the top omnibar

Author: mikescamell

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

@@ -1508,10 +1508,6 @@ class BrowserTabViewModel @Inject constructor(
 
         if (!currentBrowserViewState().browserShowing) return
 
-        if (settingsDataStore.omnibarPosition == BOTTOM) {
-            showOmniBar()
-        }
-
         canAutofillSelectCredentialsDialogCanAutomaticallyShow = true
 
         browserViewState.value = currentBrowserViewState().copy(
@@ -4003,12 +3999,6 @@ class BrowserTabViewModel @Inject constructor(
         }
     }
 
-    private fun showOmniBar() {
-        omnibarViewState.value = currentOmnibarViewState().copy(
-            navigationChange = true,
-        )
-    }
-
     fun onUserDismissedAutoCompleteInAppMessage() {
         viewModelScope.launch(dispatchers.io()) {
             autoComplete.userDismissedHistoryInAutoCompleteIAM()