[fenix] For https://github.com/mozilla-mobile/fenix/issues/1620: Hide search results when they match URL

pull/600/head
Sawyer Blatz 5 years ago committed by Emily Kager
parent 113af874ea
commit c95cc76dee

@ -166,31 +166,39 @@ class AwesomeBarView(
fun update(state: SearchFragmentState) {
view.removeAllProviders()
// Do not make suggestions based on user's current URL
if (state.query == state.session?.url) {
return
}
// Only show the shortcutEnginePicker by itself
if (state.showSearchShortcuts) {
view.addProviders(shortcutsEnginePickerProvider)
} else {
if (state.showSearchSuggestions) {
view.addProviders(
when (state.searchEngineSource) {
is SearchEngineSource.Default -> defaultSearchSuggestionProvider
is SearchEngineSource.Shortcut -> createSuggestionProviderForEngine(
state.searchEngineSource.searchEngine
)
}
)
}
view.onInputChanged(state.query)
return
}
if (state.showHistorySuggestions) {
view.addProviders(historyStorageProvider)
}
if (state.showSearchSuggestions) {
view.addProviders(
when (state.searchEngineSource) {
is SearchEngineSource.Default -> defaultSearchSuggestionProvider
is SearchEngineSource.Shortcut -> createSuggestionProviderForEngine(
state.searchEngineSource.searchEngine
)
}
)
}
if (state.showHistorySuggestions) {
view.addProviders(historyStorageProvider)
}
if (state.showBookmarkSuggestions) {
view.addProviders(bookmarksStorageSuggestionProvider)
}
if (state.showBookmarkSuggestions) {
view.addProviders(bookmarksStorageSuggestionProvider)
}
if ((container.context.asActivity() as? HomeActivity)?.browsingModeManager?.mode?.isPrivate == false) {
view.addProviders(sessionProvider)
}
if ((container.context.asActivity() as? HomeActivity)?.browsingModeManager?.mode?.isPrivate == false) {
view.addProviders(sessionProvider)
}
view.onInputChanged(state.query)

Loading…
Cancel
Save