[fenix] For https://github.com/mozilla-mobile/fenix/issues/25816: Have the previous autocomplete functionality work with the new API

The new API allows us to move from having just two autocomplete providers:
a shipped list of domains and history result to any autocomplete providers.
pull/600/head
Mugurell 2 years ago committed by mergify[bot]
parent 556c741d4f
commit 12c8d94eeb

@ -8,13 +8,13 @@ import android.content.Context
import androidx.annotation.VisibleForTesting
import androidx.core.content.ContextCompat
import androidx.lifecycle.LifecycleOwner
import mozilla.components.browser.domains.autocomplete.DomainAutocompleteProvider
import mozilla.components.browser.domains.autocomplete.BaseDomainAutocompleteProvider
import mozilla.components.browser.state.selector.normalTabs
import mozilla.components.browser.state.selector.privateTabs
import mozilla.components.browser.toolbar.BrowserToolbar
import mozilla.components.browser.toolbar.display.DisplayToolbar
import mozilla.components.concept.engine.Engine
import mozilla.components.concept.storage.HistoryStorage
import mozilla.components.concept.toolbar.AutocompleteProvider
import mozilla.components.feature.tabs.toolbar.TabCounterToolbarButton
import mozilla.components.feature.toolbar.ToolbarAutocompleteFeature
import mozilla.components.feature.toolbar.ToolbarBehaviorController
@ -77,12 +77,13 @@ abstract class ToolbarIntegration(
}
}
@Suppress("LongParameterList")
class DefaultToolbarIntegration(
context: Context,
toolbar: BrowserToolbar,
toolbarMenu: ToolbarMenu,
domainAutocompleteProvider: DomainAutocompleteProvider,
historyStorage: HistoryStorage,
domainAutocompleteProvider: BaseDomainAutocompleteProvider,
historyStorage: AutocompleteProvider,
lifecycleOwner: LifecycleOwner,
sessionId: String? = null,
isPrivate: Boolean,
@ -155,10 +156,15 @@ class DefaultToolbarIntegration(
toolbar,
engineForSpeculativeConnects,
).apply {
addDomainProvider(domainAutocompleteProvider)
if (context.settings().shouldShowHistorySuggestions) {
addHistoryStorageProvider(historyStorage)
}
updateAutocompleteProviders(
listOfNotNull(
when (context.settings().shouldShowHistorySuggestions) {
true -> historyStorage
false -> null
},
domainAutocompleteProvider,
),
)
}
}

@ -38,7 +38,7 @@ class DefaultToolbarIntegrationTest {
context = testContext,
toolbar = mockk(relaxed = true),
toolbarMenu = mockk(relaxed = true),
domainAutocompleteProvider = mockk(),
domainAutocompleteProvider = mockk(relaxed = true),
historyStorage = mockk(),
lifecycleOwner = mockk(),
sessionId = null,

Loading…
Cancel
Save