For #24708 - Remove Event.wrapper for DarkThemeSelected telemetry

pull/543/head
Alexandru2909 3 years ago committed by mergify[bot]
parent e7df75b5af
commit b57219b190

@ -5289,6 +5289,7 @@ app_theme:
A user selected Dark Theme A user selected Dark Theme
extra_keys: extra_keys:
source: source:
type: string
description: | description: |
The source from where dark theme was selected. The source can be The source from where dark theme was selected. The source can be
'SETTINGS' or 'ONBOARDING' 'SETTINGS' or 'ONBOARDING'

@ -7,7 +7,6 @@ package org.mozilla.fenix.components.metrics
import mozilla.components.browser.state.search.SearchEngine import mozilla.components.browser.state.search.SearchEngine
import mozilla.components.feature.top.sites.TopSite import mozilla.components.feature.top.sites.TopSite
import org.mozilla.fenix.GleanMetrics.Addons import org.mozilla.fenix.GleanMetrics.Addons
import org.mozilla.fenix.GleanMetrics.AppTheme
import org.mozilla.fenix.GleanMetrics.Autoplay import org.mozilla.fenix.GleanMetrics.Autoplay
import org.mozilla.fenix.GleanMetrics.ContextMenu import org.mozilla.fenix.GleanMetrics.ContextMenu
import org.mozilla.fenix.GleanMetrics.Events import org.mozilla.fenix.GleanMetrics.Events
@ -246,13 +245,6 @@ sealed class Event {
get() = mapOf(Events.performedSearchKeys.source to eventSource.sourceLabel) get() = mapOf(Events.performedSearchKeys.source to eventSource.sourceLabel)
} }
data class DarkThemeSelected(val source: Source) : Event() {
enum class Source { SETTINGS }
override val extras: Map<AppTheme.darkThemeSelectedKeys, String>?
get() = mapOf(AppTheme.darkThemeSelectedKeys.source to source.name)
}
data class SearchWithAds(val providerName: String) : Event() { data class SearchWithAds(val providerName: String) : Event() {
val label: String val label: String
get() = providerName get() = providerName

@ -10,7 +10,6 @@ import mozilla.components.service.glean.private.NoExtraKeys
import mozilla.components.support.base.log.logger.Logger import mozilla.components.support.base.log.logger.Logger
import org.mozilla.fenix.GleanMetrics.Addons import org.mozilla.fenix.GleanMetrics.Addons
import org.mozilla.fenix.GleanMetrics.AndroidAutofill import org.mozilla.fenix.GleanMetrics.AndroidAutofill
import org.mozilla.fenix.GleanMetrics.AppTheme
import org.mozilla.fenix.GleanMetrics.Autoplay import org.mozilla.fenix.GleanMetrics.Autoplay
import org.mozilla.fenix.GleanMetrics.Awesomebar import org.mozilla.fenix.GleanMetrics.Awesomebar
import org.mozilla.fenix.GleanMetrics.BrowserSearch import org.mozilla.fenix.GleanMetrics.BrowserSearch
@ -220,10 +219,6 @@ private val Event.wrapper: EventWrapper<*>?
{ Pocket.homeRecsCategoryClicked.record(it) }, { Pocket.homeRecsCategoryClicked.record(it) },
{ Pocket.homeRecsCategoryClickedKeys.valueOf(it) } { Pocket.homeRecsCategoryClickedKeys.valueOf(it) }
) )
is Event.DarkThemeSelected -> EventWrapper(
{ AppTheme.darkThemeSelected.record(it) },
{ AppTheme.darkThemeSelectedKeys.valueOf(it) }
)
is Event.AddonsOpenInSettings -> EventWrapper<NoExtraKeys>( is Event.AddonsOpenInSettings -> EventWrapper<NoExtraKeys>(
{ Addons.openAddonsInSettings.record(it) } { Addons.openAddonsInSettings.record(it) }
) )

@ -12,11 +12,10 @@ import androidx.appcompat.app.AppCompatDelegate
import androidx.preference.PreferenceFragmentCompat import androidx.preference.PreferenceFragmentCompat
import androidx.preference.SwitchPreference import androidx.preference.SwitchPreference
import org.mozilla.fenix.FeatureFlags import org.mozilla.fenix.FeatureFlags
import org.mozilla.fenix.GleanMetrics.AppTheme
import org.mozilla.fenix.GleanMetrics.ToolbarSettings import org.mozilla.fenix.GleanMetrics.ToolbarSettings
import org.mozilla.fenix.R import org.mozilla.fenix.R
import org.mozilla.fenix.components.metrics.Event
import org.mozilla.fenix.components.toolbar.ToolbarPosition import org.mozilla.fenix.components.toolbar.ToolbarPosition
import org.mozilla.fenix.ext.components
import org.mozilla.fenix.ext.requireComponents import org.mozilla.fenix.ext.requireComponents
import org.mozilla.fenix.ext.settings import org.mozilla.fenix.ext.settings
import org.mozilla.fenix.ext.showToolbar import org.mozilla.fenix.ext.showToolbar
@ -85,9 +84,9 @@ class CustomizationFragment : PreferenceFragmentCompat() {
private fun bindDarkTheme() { private fun bindDarkTheme() {
radioDarkTheme = requirePreference(R.string.pref_key_dark_theme) radioDarkTheme = requirePreference(R.string.pref_key_dark_theme)
radioDarkTheme.onClickListener { radioDarkTheme.onClickListener {
requireContext().components.analytics.metrics.track( AppTheme.darkThemeSelected.record(
Event.DarkThemeSelected( AppTheme.DarkThemeSelectedExtra(
Event.DarkThemeSelected.Source.SETTINGS "SETTINGS"
) )
) )
setNewTheme(AppCompatDelegate.MODE_NIGHT_YES) setNewTheme(AppCompatDelegate.MODE_NIGHT_YES)

Loading…
Cancel
Save