Enable recording of history metadata for all builds

This allows recording part of history metadata to ride the trains.
The UI features are still guarded by the secret settings flag (or,
enabled on debug builds).
upstream-sync
Grisha Kruglov 3 years ago committed by mergify[bot]
parent 8b4190a429
commit fc7a92bf10

@ -45,9 +45,9 @@ object FeatureFlags {
val showRecentTabsFeature = Config.channel.isNightlyOrDebug val showRecentTabsFeature = Config.channel.isNightlyOrDebug
/** /**
* Enables recording of history metadata. * Enables UI features based on history metadata.
*/ */
val historyMetadataFeature = Config.channel.isDebug val historyMetadataUIFeature = Config.channel.isDebug
/** /**
* Enables the recently saved bookmarks feature in the home screen. * Enables the recently saved bookmarks feature in the home screen.

@ -272,9 +272,7 @@ class BrowserFragment : BaseBrowserFragment(), UserInteractionHandler {
override fun onStop() { override fun onStop() {
super.onStop() super.onStop()
updateLastBrowseActivity() updateLastBrowseActivity()
if (requireContext().settings().historyMetadataFeature) { updateHistoryMetadata()
updateHistoryMetadata()
}
pwaOnboardingObserver?.stop() pwaOnboardingObserver?.stop()
} }

@ -208,13 +208,10 @@ class Core(
RecordingDevicesMiddleware(context), RecordingDevicesMiddleware(context),
PromptMiddleware(), PromptMiddleware(),
AdsTelemetryMiddleware(adsTelemetry), AdsTelemetryMiddleware(adsTelemetry),
LastMediaAccessMiddleware() LastMediaAccessMiddleware(),
HistoryMetadataMiddleware(historyMetadataService)
) )
if (context.settings().historyMetadataFeature) {
middlewareList += HistoryMetadataMiddleware(historyMetadataService)
}
BrowserStore( BrowserStore(
middleware = middlewareList + EngineMiddleware.create(engine) middleware = middlewareList + EngineMiddleware.create(engine)
).apply { ).apply {

@ -270,7 +270,7 @@ class HomeFragment : Fragment() {
) )
} }
if (requireContext().settings().historyMetadataFeature) { if (requireContext().settings().historyMetadataUIFeature) {
historyMetadataFeature.set( historyMetadataFeature.set(
feature = HistoryMetadataFeature( feature = HistoryMetadataFeature(
homeStore = homeFragmentStore, homeStore = homeFragmentStore,

@ -251,7 +251,7 @@ class AwesomeBarView(
val providersToAdd = mutableSetOf<AwesomeBar.SuggestionProvider>() val providersToAdd = mutableSetOf<AwesomeBar.SuggestionProvider>()
if (state.showHistorySuggestions) { if (state.showHistorySuggestions) {
if (activity.settings().historyMetadataFeature) { if (activity.settings().historyMetadataUIFeature) {
providersToAdd.add(combinedHistoryProvider) providersToAdd.add(combinedHistoryProvider)
} else { } else {
providersToAdd.add(historyStorageProvider) providersToAdd.add(historyStorageProvider)
@ -285,7 +285,7 @@ class AwesomeBarView(
providersToRemove.add(shortcutsEnginePickerProvider) providersToRemove.add(shortcutsEnginePickerProvider)
if (!state.showHistorySuggestions) { if (!state.showHistorySuggestions) {
if (activity.settings().historyMetadataFeature) { if (activity.settings().historyMetadataUIFeature) {
providersToRemove.add(combinedHistoryProvider) providersToRemove.add(combinedHistoryProvider)
} else { } else {
providersToRemove.add(historyStorageProvider) providersToRemove.add(historyStorageProvider)

@ -36,7 +36,7 @@ class SecretSettingsFragment : PreferenceFragmentCompat() {
requirePreference<SwitchPreference>(R.string.pref_key_history_metadata_feature).apply { requirePreference<SwitchPreference>(R.string.pref_key_history_metadata_feature).apply {
isVisible = true isVisible = true
isChecked = context.settings().historyMetadataFeature isChecked = context.settings().historyMetadataUIFeature
onPreferenceChangeListener = object : SharedPreferenceUpdater() { onPreferenceChangeListener = object : SharedPreferenceUpdater() {
override fun onPreferenceChange(preference: Preference, newValue: Any?): Boolean { override fun onPreferenceChange(preference: Preference, newValue: Any?): Boolean {
val result = super.onPreferenceChange(preference, newValue) val result = super.onPreferenceChange(preference, newValue)

@ -1101,10 +1101,10 @@ class Settings(private val appContext: Context) : PreferencesHolder {
default = false default = false
) )
var historyMetadataFeature by featureFlagPreference( var historyMetadataUIFeature by featureFlagPreference(
appContext.getPreferenceKey(R.string.pref_key_history_metadata_feature), appContext.getPreferenceKey(R.string.pref_key_history_metadata_feature),
default = FeatureFlags.historyMetadataFeature, default = FeatureFlags.historyMetadataUIFeature,
featureFlag = FeatureFlags.historyMetadataFeature || isHistoryMetadataEnabled featureFlag = FeatureFlags.historyMetadataUIFeature || isHistoryMetadataEnabled
) )
/** /**

Loading…
Cancel
Save