Omnibar Refactor: Use LegacyOmnibar (#5073)

Task/Issue URL:
https://app.asana.com/0/1174433894299346/1208319036523674

### Description
This PR continues with the refactoring, moving logic from the BTF into
the Omnibar facade.

### Steps to test this PR
Smoke test of the app, chrome customtab, etc..

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

@@ -856,6 +856,7 @@ class BrowserTabViewModel @Inject constructor(
  AppPixelName.AUTOCOMPLETE_BOOKMARK_SELECTION
  }
  }
+
  is AutoCompleteSearchSuggestion -> if (suggestion.isUrl) AUTOCOMPLETE_SEARCH_WEBSITE_SELECTION else AUTOCOMPLETE_SEARCH_PHRASE_SELECTION
  is AutoCompleteHistorySuggestion -> AUTOCOMPLETE_HISTORY_SITE_SELECTION
  is AutoCompleteHistorySearchSuggestion -> AUTOCOMPLETE_HISTORY_SEARCH_SELECTION
@@ -895,7 +896,10 @@ class BrowserTabViewModel @Inject constructor(
  }
  }
 
- fun onRemoveSearchSuggestionConfirmed(suggestion: AutoCompleteSuggestion, omnibarText: String) {
+ fun onRemoveSearchSuggestionConfirmed(
+ suggestion: AutoCompleteSuggestion,
+ omnibarText: String,
+ ) {
  appCoroutineScope.launch(dispatchers.io()) {
  pixel.fire(AUTOCOMPLETE_RESULT_DELETED)
  pixel.fire(AUTOCOMPLETE_RESULT_DELETED_DAILY, type = Daily())
@@ -904,9 +908,11 @@ class BrowserTabViewModel @Inject constructor(
  is AutoCompleteHistorySuggestion -> {
  history.removeHistoryEntryByUrl(suggestion.url)
  }
+
  is AutoCompleteHistorySearchSuggestion -> {
  history.removeHistoryEntryByQuery(suggestion.phrase)
  }
+
  else -> {}
  }
  withContext(dispatchers.main()) {
@@ -1190,7 +1196,10 @@ class BrowserTabViewModel @Inject constructor(
  }
  }
 
- fun urlUnchangedForExternalLaunchPurposes(oldUrl: String?, newUrl: String): Boolean {
+ fun urlUnchangedForExternalLaunchPurposes(
+ oldUrl: String?,
+ newUrl: String,
+ ): Boolean {
  if (oldUrl == null) return false
  fun normalizeUrl(url: String): String {
  val regex = Regex("^(https?://)?(www\\.)?")
@@ -1202,6 +1211,7 @@ class BrowserTabViewModel @Inject constructor(
 
  return normalizedUrl
  }
+
  val normalizedOldUrl = normalizeUrl(oldUrl)
  val normalizedNewUrl = normalizeUrl(newUrl)
  return normalizedOldUrl == normalizedNewUrl
@@ -1322,6 +1332,7 @@ class BrowserTabViewModel @Inject constructor(
  }
  }
  }
+
  is WebNavigationStateChange.PageCleared -> pageCleared()
  is WebNavigationStateChange.UrlUpdated -> {
  val uri = stateChange.url.toUri()
@@ -1339,6 +1350,7 @@ class BrowserTabViewModel @Inject constructor(
  }
  }
  }
+
  is WebNavigationStateChange.PageNavigationCleared -> disableUserNavigation()
  else -> {}
  }
@@ -2413,7 +2425,10 @@ class BrowserTabViewModel @Inject constructor(
  }
  }
 
- private suspend fun addToAllowList(domain: String, clickedFromCustomTab: Boolean) {
+ private suspend fun addToAllowList(
+ domain: String,
+ clickedFromCustomTab: Boolean,
+ ) {
  val pixelParams = privacyProtectionsPopupExperimentExternalPixels.getPixelParams()
  if (clickedFromCustomTab) {
  pixel.fire(CustomTabPixelNames.CUSTOM_TABS_MENU_DISABLE_PROTECTIONS_ALLOW_LIST_ADD)
@@ -2428,7 +2443,10 @@ class BrowserTabViewModel @Inject constructor(
  }
  }
 
- private suspend fun removeFromAllowList(domain: String, clickedFromCustomTab: Boolean) {
+ private suspend fun removeFromAllowList(
+ domain: String,
+ clickedFromCustomTab: Boolean,
+ ) {
  val pixelParams = privacyProtectionsPopupExperimentExternalPixels.getPixelParams()
  if (clickedFromCustomTab) {
  pixel.fire(CustomTabPixelNames.CUSTOM_TABS_MENU_DISABLE_PROTECTIONS_ALLOW_LIST_REMOVE)
@@ -2515,6 +2533,10 @@ class BrowserTabViewModel @Inject constructor(
 
  fun userFindingInPage(searchTerm: String) {
  val currentViewState = currentFindInPageViewState()
+ if (!currentViewState.visible && searchTerm.isEmpty()) {
+ return
+ }
+
  var findInPage = currentViewState.copy(visible = true, searchTerm = searchTerm)
  if (searchTerm.isEmpty()) {
  findInPage = findInPage.copy(showNumberMatches = false)
@@ -3012,7 +3034,10 @@ class BrowserTabViewModel @Inject constructor(
  }
  }
 
- private fun sameOrigin(firstUrl: String, secondUrl: String): Boolean {
+ private fun sameOrigin(
+ firstUrl: String,
+ secondUrl: String,
+ ): Boolean {
  return kotlin.runCatching {
  val firstUri = Uri.parse(firstUrl)
  val secondUri = Uri.parse(secondUrl)
@@ -3359,6 +3384,7 @@ class BrowserTabViewModel @Inject constructor(
  }
  }
  }
+
  else -> {}
  }
  }
@@ -3450,7 +3476,10 @@ class BrowserTabViewModel @Inject constructor(
  }
  }
 
- fun onSSLCertificateWarningAction(action: Action, url: String) {
+ fun onSSLCertificateWarningAction(
+ action: Action,
+ url: String,
+ ) {
  when (action) {
  is Action.Shown -> {
  when (action.errorType) {
@@ -3547,6 +3576,7 @@ class BrowserTabViewModel @Inject constructor(
  }
  null
  }
+
  else -> null
  }
  }
@@ -3642,7 +3672,11 @@ class BrowserTabViewModel @Inject constructor(
  }
  }
 
- fun saveReplyProxyForBlobDownload(originUrl: String, replyProxy: JavaScriptReplyProxy, locationHref: String? = null) {
+ fun saveReplyProxyForBlobDownload(
+ originUrl: String,
+ replyProxy: JavaScriptReplyProxy,
+ locationHref: String? = null,
+ ) {
  appCoroutineScope.launch(dispatchers.io()) { // FF check has disk IO
  if (androidBrowserConfig.fixBlobDownloadWithIframes().isEnabled()) {
  val frameProxies = fixedReplyProxyMap[originUrl]?.toMutableMap() ?: mutableMapOf()
@@ -3706,7 +3740,11 @@ class BrowserTabViewModel @Inject constructor(
 
  fun hasOmnibarPositionChanged(currentPosition: OmnibarPosition): Boolean = settingsDataStore.omnibarPosition != currentPosition
 
- private fun firePixelBasedOnCurrentUrl(emptyUrlPixel: AppPixelName, duckDuckGoQueryUrlPixel: AppPixelName, websiteUrlPixel: AppPixelName) {
+ private fun firePixelBasedOnCurrentUrl(
+ emptyUrlPixel: AppPixelName,
+ duckDuckGoQueryUrlPixel: AppPixelName,
+ websiteUrlPixel: AppPixelName,
+ ) {
  val text = url.orEmpty()
  if (text.isEmpty()) {
  pixel.fire(emptyUrlPixel)