For #15369: Add synced tabs usage metrics.

mozilla-mobile · Nov 24, 2020 · b4e7d64 · b4e7d64
1 parent 4615aba
commit b4e7d64
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
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
@@ -183,6 +183,8 @@ sealed class Event {
 
     object TabSettingsOpened : Event()
 
+    object SyncedTabOpened : Event()
+
     // Interaction events with extras
 
     data class TopSiteSwipeCarousel(val page: Int) : 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
@@ -659,6 +659,12 @@ private val Event.wrapper: EventWrapper<*>?
             { ProgressiveWebApp.backgroundKeys.valueOf(it) }
         )
 
+        Event.SyncedTabOpened -> EventWrapper<NoExtraKeys>(
+            { Metrics.syncedTabsOpenedCount.add(1) }
+        ).also {
+            Metrics.hasOpenedSyncedTabs.set(true)
+        }
+
         Event.MasterPasswordMigrationDisplayed -> EventWrapper<NoExtraKeys>(
             { MasterPassword.displayed.record(it) }
         )

diff --git a/app/src/main/java/org/mozilla/fenix/tabtray/TabTrayController.kt b/app/src/main/java/org/mozilla/fenix/tabtray/TabTrayController.kt
@@ -205,6 +205,7 @@ class DefaultTabTrayController(
             newTab = true,
             from = BrowserDirection.FromTabTray
         )
+        metrics.track(Event.SyncedTabOpened)
     }
 
     @OptIn(ExperimentalCoroutinesApi::class)