For #800 - Adds metrics for the default search engine

mozilla-mobile · Apr 12, 2019 · a2482cc · a2482cc
1 parent bc1b7e0
commit a2482cc
Show file tree

Hide file tree

Showing 2 changed files with 54 additions and 1 deletion.
diff --git a/app/metrics.yaml b/app/metrics.yaml
@@ -326,4 +326,42 @@ metrics:
     notification_emails:
     - telemetry-client-dev@mozilla.com
     - fenix-core@mozilla.com
-    expires: "2020-03-01"
+    expires: "2020-03-01"
+
+search.default_engine:
+  code:
+    type: string
+    description: The search code for the search engine
+    send_in_pings:
+    - metrics
+    bugs:
+    - 800
+    data_reviews:
+    - TBD
+    notification_emails:
+    - fenix-core@mozilla.com
+    expires: "2020-03-01"
+  name:
+    type: string
+    description: The name of the search engine
+    send_in_pings:
+    - metrics
+    bugs:
+    - 800
+    data_reviews:
+    - TBD
+    notification_emails:
+    - fenix-core@mozilla.com
+    expires: "2020-03-01"
+  submission_url:
+    type: string
+    description: The full url we use to query the serach engine
+    send_in_pings:
+    - metrics
+    bugs:
+    - 800
+    data_reviews:
+    - TBD
+    notification_emails:
+    - fenix-core@mozilla.com
+    expires: "2020-03-01"
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
@@ -13,6 +13,8 @@ import org.mozilla.fenix.GleanMetrics.FindInPage
 import org.mozilla.fenix.GleanMetrics.ContextMenu
 import org.mozilla.fenix.GleanMetrics.QuickActionSheet
 import org.mozilla.fenix.GleanMetrics.Metrics
+import org.mozilla.fenix.GleanMetrics.SearchDefaultEngine
+import org.mozilla.fenix.ext.components
 
 private class EventWrapper<T : Enum<T>>(
     private val recorder: ((Map<T, String>?) -> Unit),
@@ -119,6 +121,19 @@ class GleanMetricsService(private val context: Context) : MetricsService {
             defaultBrowser.set(Browsers.all(context).isDefaultBrowser)
         }
 
+        SearchDefaultEngine.apply {
+            val defaultEngine = context
+                .components
+                .search
+                .searchEngineManager
+                .defaultSearchEngine ?: return@apply
+
+            code.set(defaultEngine.identifier)
+            name.set(defaultEngine.name)
+            submissionUrl.set(defaultEngine.buildSearchUrl(""))
+        }
+
+
         initialized = true
     }