From aac9149bb52d1e482c58b6f3288d60ce5f94f2ad Mon Sep 17 00:00:00 2001 From: Michael Comella Date: Thu, 24 Jun 2021 12:02:55 -0700 Subject: [PATCH] [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/20127: fix app_received_intent extras. Apparently, it had been using the wrong mapping this whole time. I can't wrap my head around fenix's glean wrapper so I'm not sure if this resulted in bugs or if my change changes the behavior. However, I don't think anyone is using this probe so I'm requesting to remove it. If we don't end up removing it, then we can check for bugs/changes in behavior. This isn't ideal but I'd rather not spend the time on it if we don't have to. --- .../main/java/org/mozilla/fenix/components/metrics/Event.kt | 4 ++-- 1 file changed, 2 insertions(+), 2 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 index d48165b0f4..e3fe6a9f03 100644 --- a/app/src/main/java/org/mozilla/fenix/components/metrics/Event.kt +++ b/app/src/main/java/org/mozilla/fenix/components/metrics/Event.kt @@ -403,8 +403,8 @@ sealed class Event { enum class Source { APP_ICON, LINK, CUSTOM_TAB, UNKNOWN } - override val extras: Map? - get() = hashMapOf(Events.appOpenedAllStartupKeys.source to source.name) + override val extras: Map? + get() = hashMapOf(Events.appReceivedIntentKeys.source to source.name) } data class CollectionSaveButtonPressed(val fromScreen: String) : Event() {