No issue: Rename Event for naming consistency.

(cherry picked from commit 8b3a878062)
pull/420/head
mcarare 3 years ago committed by Arturo Mejia
parent 106aa6237d
commit 44de36e302

@ -226,7 +226,7 @@ sealed class Event {
object OpenedTabSuggestionClicked : Event()
// Set default browser experiment metrics
object SetDefaultBrowserClicked : Event()
object SetDefaultBrowserNewTabClicked : Event()
object CloseExperimentCardClicked : Event()
object ToolbarMenuShown : Event()
object SetDefaultBrowserToolbarMenuClicked : Event()

@ -833,7 +833,7 @@ private val Event.wrapper: EventWrapper<*>?
is Event.CloseExperimentCardClicked -> EventWrapper<NoExtraKeys>(
{ SetDefaultNewtabExperiment.closeExperimentCardClicked.record(it) }
)
is Event.SetDefaultBrowserClicked -> EventWrapper<NoExtraKeys>(
is Event.SetDefaultBrowserNewTabClicked -> EventWrapper<NoExtraKeys>(
{ SetDefaultNewtabExperiment.setDefaultBrowserClicked.record(it) }
)
is Event.SetDefaultBrowserSettingsScreenClicked -> EventWrapper<NoExtraKeys>(

@ -567,7 +567,7 @@ class DefaultSessionControlController(
override fun handleSetDefaultBrowser() {
settings.userDismissedExperimentCard = true
metrics.track(Event.SetDefaultBrowserClicked)
metrics.track(Event.SetDefaultBrowserNewTabClicked)
activity.openSetDefaultBrowserOption()
}

Loading…
Cancel
Save