For #4123: Adds telemetry for quick action open in app (#4629)

app/metrics.yaml

@@ -314,6 +314,17 @@ quick_action_sheet:
     notification_emails:
     - fenix-core@mozilla.com
     expires: "2020-03-01"
+  open_app_tapped:
+    type: event
+    description: >
+      A user tapped the open in app button
+    bugs:
+      - 1195
+    data_reviews:
+      - https://github.com/mozilla-mobile/fenix/pull/4629
+    notification_emails:
+      - fenix-core@mozilla.com
+    expires: "2020-03-01"
 
 metrics:
   default_browser:

app/src/main/java/org/mozilla/fenix/components/metrics/GleanMetricsService.kt

@@ -122,6 +122,9 @@ private val Event.wrapper
         is Event.QuickActionSheetDownloadTapped -> EventWrapper<NoExtraKeys>(
             { QuickActionSheet.downloadTapped.record(it) }
         )
+        is Event.QuickActionSheetOpenInAppTapped -> EventWrapper<NoExtraKeys>(
+            { QuickActionSheet.openAppTapped.record(it) }
+        )
         is Event.OpenedBookmarkInNewTab -> EventWrapper<NoExtraKeys>(
             { BookmarksManagement.openInNewTab.record(it) }
         )

app/src/main/java/org/mozilla/fenix/components/metrics/Metrics.kt

@@ -73,6 +73,7 @@ sealed class Event {
     object QuickActionSheetShareTapped : Event()
     object QuickActionSheetBookmarkTapped : Event()
     object QuickActionSheetDownloadTapped : Event()
+    object QuickActionSheetOpenInAppTapped : Event()
     object CustomTabsClosed : Event()
     object CustomTabsActionTapped : Event()
     object CustomTabsMenuOpened : Event()

app/src/main/java/org/mozilla/fenix/quickactionsheet/QuickActionSheetController.kt

@@ -53,6 +53,8 @@ class DefaultQuickActionSheetController(
     }
 
     override fun handleOpenLink() {
+        context.metrics.track(Event.QuickActionSheetOpenInAppTapped)
+
         val getRedirect = appLinksUseCases.appLinkRedirect
         val redirect = currentSession.let {
             getRedirect.invoke(it.url)

buildSrc/src/main/java/Dependencies.kt

@@ -205,7 +205,7 @@ object Deps {
     const val robolectric = "org.robolectric:robolectric:${Versions.robolectric}"
     const val fragment_testing = "androidx.fragment:fragment-testing:${Versions.androidx_fragment}"
     const val androidx_junit = "androidx.test.ext:junit:${Versions.androidx_test_ext}"
-    const val androidx_test_core ="androidx.test:core:${Versions.androidx_testing}"
+    const val androidx_test_core = "androidx.test:core:${Versions.androidx_testing}"
 
     const val fenix_megazord = "org.mozilla.appservices:fenix-megazord:${Versions.mozilla_appservices}"
     const val fenix_megazord_forUnitTests = "org.mozilla.appservices:fenix-megazord-forUnitTests:${Versions.mozilla_appservices}"

docs/metrics.md

@@ -314,6 +314,14 @@ tracking_protection</td>
         <td></td>
         <td>2020-03-01</td>
     </tr>
+    <tr>
+        <td>open_app_tapped</td>
+        <td>event</td>
+        <td>A user tapped the open in app button</td>
+        <td><a href="https://github.com/mozilla-mobile/fenix/pull/4629">link</a></td>
+        <td></td>
+        <td>2020-03-01</td>
+    </tr>
 </table>
 </pre>