From 9251597e8407a73f2b7b3e148a77a3cfd68bbdd2 Mon Sep 17 00:00:00 2001 From: Sawyer Blatz Date: Thu, 18 Jul 2019 13:36:52 -0700 Subject: [PATCH] For #2362: Fixes telemetry for history (#4156) --- .../java/org/mozilla/fenix/library/history/HistoryFragment.kt | 3 +++ 1 file changed, 3 insertions(+) diff --git a/app/src/main/java/org/mozilla/fenix/library/history/HistoryFragment.kt b/app/src/main/java/org/mozilla/fenix/library/history/HistoryFragment.kt index 914a5ff1d..b30844a35 100644 --- a/app/src/main/java/org/mozilla/fenix/library/history/HistoryFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/library/history/HistoryFragment.kt @@ -89,6 +89,7 @@ class HistoryFragment : Fragment(), BackHandler { lifecycleScope.launch { val storage = context?.components?.core?.historyStorage for (item in items) { + context?.components?.analytics?.metrics?.track(Event.HistoryItemRemoved) storage?.deleteVisit(item.url, item.visitedAt) } reloadData() @@ -173,6 +174,7 @@ class HistoryFragment : Fragment(), BackHandler { (historyStore.state.mode as? HistoryState.Mode.Editing)?.selectedItems ?: listOf() requireComponents.useCases.tabsUseCases.addTab.let { useCase -> for (selectedItem in selectedHistory) { + requireComponents.analytics.metrics.track(Event.HistoryItemOpened) useCase.invoke(selectedItem.url) } } @@ -192,6 +194,7 @@ class HistoryFragment : Fragment(), BackHandler { (historyStore.state.mode as? HistoryState.Mode.Editing)?.selectedItems ?: listOf() requireComponents.useCases.tabsUseCases.addPrivateTab.let { useCase -> for (selectedItem in selectedHistory) { + requireComponents.analytics.metrics.track(Event.HistoryItemOpened) useCase.invoke(selectedItem.url) } }