For mozilla-mobile#24715: Remove obsolete addon installed metric.

mcarare · Apr 12, 2022 · ffccc69 · ffccc69
1 parent c617696
commit ffccc69
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 4 deletions.
diff --git a/app/src/main/java/org/mozilla/fenix/addons/AddonsManagementFragment.kt b/app/src/main/java/org/mozilla/fenix/addons/AddonsManagementFragment.kt
@@ -231,7 +231,6 @@ class AddonsManagementFragment : Fragment(R.layout.fragment_add_ons_management)
 
     private fun showInstallationDialog(addon: Addon) {
         if (!isInstallationInProgress && !hasExistingAddonInstallationDialogFragment()) {
-            requireComponents.analytics.metrics.track(Event.AddonInstalled(addon.id))
             val context = requireContext()
             val addonCollectionProvider = context.components.addonCollectionProvider
 

diff --git a/app/src/main/java/org/mozilla/fenix/components/metrics/Event.kt b/app/src/main/java/org/mozilla/fenix/components/metrics/Event.kt
@@ -269,8 +269,6 @@ sealed class Event {
         }
     }
 
-    data class AddonInstalled(val addonId: String) : Event()
-
     object AutoPlaySettingVisited : Event()
 
     data class AutoPlaySettingChanged(val setting: AutoplaySetting) : Event() {

diff --git a/app/src/main/java/org/mozilla/fenix/components/metrics/GleanMetricsService.kt b/app/src/main/java/org/mozilla/fenix/components/metrics/GleanMetricsService.kt
@@ -498,7 +498,6 @@ private val Event.wrapper: EventWrapper<*>?
         is Event.InteractWithSearchURLArea -> null
         is Event.ClearedPrivateData -> null
         is Event.DismissedOnboarding -> null
-        is Event.AddonInstalled -> null
         is Event.SearchWidgetInstalled -> null
     }
-Original file line number
+Diff line change
@@ Expand Up / @@ -269,8 +269,6 @@ sealed class Event { @@
             }
         }
-        data class AddonInstalled(val addonId: String) : Event()
         object AutoPlaySettingVisited : Event()
         data class AutoPlaySettingChanged(val setting: AutoplaySetting) : Event() {
@@ Expand Down @@