For mozilla-mobile#7249 Create a search widget

iorgamgabriel · Aug 9, 2022 · 933883f · 933883f
1 parent 0794dea
commit 933883f
Show file tree

Hide file tree

Showing 5 changed files with 78 additions and 9 deletions.
diff --git a/app/metrics.yaml b/app/metrics.yaml
@@ -1900,3 +1900,57 @@ metrics:
     metadata:
       tags:
         - Performance
+  search_widget_installed:
+    type: boolean
+    lifetime: application
+    description: |
+      Whether or not the search widget is installed
+    send_in_pings:
+      - metrics
+    bugs:
+      - https://github.com/mozilla-mobile/fenix/issues/
+    data_reviews:
+      - https://github.com/mozilla-mobile/fenix/pull/
+    data_sensitivity:
+      - interaction
+    notification_emails:
+      - [email protected]
+    expires: 119
+    metadata:
+      tags:
+        - Search
+
+search_widget:
+  new_tab_button:
+    type: event
+    description: |
+      A user pressed anywhere from the Focus logo until the start of the
+      microphone icon, opening a new tab search screen.
+    bugs:
+      - https://github.com/mozilla-mobile/fenix/issues/
+    data_reviews:
+      - https://github.com/mozilla-mobile/fenix/pull/
+    data_sensitivity:
+      - interaction
+    notification_emails:
+      - [email protected]
+    expires: 119
+    metadata:
+      tags:
+        - Search
+  voice_button:
+    type: event
+    description: |
+      A user pressed the microphone icon, opening a new voice search screen.
+    bugs:
+      - https://github.com/mozilla-mobile/fenix/issues/
+    data_reviews:
+      - https://github.com/mozilla-mobile/fenix/pull/
+    data_sensitivity:
+      - interaction
+    notification_emails:
+      - [email protected]
+    expires: 119
+    metadata:
+      tags:
+        - Search
diff --git a/app/src/main/java/org/mozilla/focus/activity/IntentReceiverActivity.kt b/app/src/main/java/org/mozilla/focus/activity/IntentReceiverActivity.kt
@@ -8,7 +8,9 @@ import android.app.Activity
 import android.content.Intent
 import android.os.Bundle
 import mozilla.components.feature.intent.ext.sanitize
+import mozilla.components.service.glean.private.NoExtras
 import mozilla.components.support.utils.toSafeIntent
+import org.mozilla.focus.GleanMetrics.SearchWidget
 import org.mozilla.focus.ext.components
 import org.mozilla.focus.session.IntentProcessor
 import org.mozilla.focus.utils.SupportUtils
@@ -25,7 +27,9 @@ class IntentReceiverActivity : Activity() {
         super.onCreate(savedInstanceState)
 
         val intent = intent.sanitize().toSafeIntent()
-
+        if (intent.getBooleanExtra(SEARCH_WIDGET, false)) {
+            SearchWidget.newTabButton.record(NoExtras())
+        }
         if (intent.dataString.equals(SupportUtils.OPEN_WITH_DEFAULT_BROWSER_URL)) {
             dispatchNormalIntent()
             return
@@ -61,4 +65,8 @@ class IntentReceiverActivity : Activity() {
 
         startActivity(intent)
     }
+
+    companion object {
+        const val SEARCH_WIDGET = "search_widget"
+    }
 }
diff --git a/app/src/main/java/org/mozilla/focus/searchwidget/VoiceSearchActivity.kt b/app/src/main/java/org/mozilla/focus/searchwidget/VoiceSearchActivity.kt
@@ -8,6 +8,8 @@ import android.content.Intent
 import mozilla.components.feature.search.widget.BaseVoiceSearchActivity
 import mozilla.components.support.locale.LocaleManager
 import mozilla.components.support.locale.LocaleManager.getCurrentLocale
+import mozilla.telemetry.glean.private.NoExtras
+import org.mozilla.focus.GleanMetrics.SearchWidget
 import org.mozilla.focus.activity.IntentReceiverActivity
 import java.util.Locale
 
@@ -18,15 +20,15 @@ class VoiceSearchActivity : BaseVoiceSearchActivity() {
             ?: LocaleManager.getSystemDefault()
     }
 
-    override fun recordVoiceButtonTelemetry() {
-        // TO DO add
-    }
-
-    override fun startIntentAfterVoiceSearch(spokenText: String?) {
+    override fun onSpeechRecognitionEnded(spokenText: String) {
         val intent = Intent(this, IntentReceiverActivity::class.java)
         intent.action = Intent.ACTION_SEND
         intent.putExtra(SPEECH_PROCESSING, spokenText)
         intent.putExtra(Intent.EXTRA_TEXT, spokenText)
         startActivity(intent)
     }
+
+    override fun onSpeechRecognitionStarted() {
+        SearchWidget.voiceButton.record(NoExtras())
+    }
 }
diff --git a/app/src/main/java/org/mozilla/focus/telemetry/GleanMetricsService.kt b/app/src/main/java/org/mozilla/focus/telemetry/GleanMetricsService.kt
@@ -24,6 +24,7 @@ import org.mozilla.focus.Components
 import org.mozilla.focus.GleanMetrics.Browser
 import org.mozilla.focus.GleanMetrics.GleanBuildInfo
 import org.mozilla.focus.GleanMetrics.LegacyIds
+import org.mozilla.focus.GleanMetrics.Metrics
 import org.mozilla.focus.GleanMetrics.MozillaProducts
 import org.mozilla.focus.GleanMetrics.Pings
 import org.mozilla.focus.GleanMetrics.Preferences
@@ -75,7 +76,7 @@ class GleanMetricsService(context: Context) : MetricsService {
         GlobalScope.launch(IO) {
 
             // Wait for preferences to be collected before we send the activation ping.
-            collectPrefMetrics(components, settings, context).await()
+            collectPrefMetricsAsync(components, settings, context).await()
 
             // Set the client ID in Glean as part of the deletion-request.
             LegacyIds.clientId.set(UUID.fromString(TelemetryWrapper.clientId))
@@ -90,7 +91,7 @@ class GleanMetricsService(context: Context) : MetricsService {
         }
     }
 
-    private fun collectPrefMetrics(
+    private fun collectPrefMetricsAsync(
         components: Components,
         settings: Settings,
         context: Context
@@ -100,6 +101,8 @@ class GleanMetricsService(context: Context) : MetricsService {
         val isFenixDefaultBrowser = FenixProductDetector.isFenixDefaultBrowser(installedBrowsers.defaultBrowser)
         val isFocusDefaultBrowser = installedBrowsers.isDefaultBrowser
 
+        Metrics.searchWidgetInstalled.set(settings.searchWidgetInstalled)
+
         Browser.isDefault.set(isFocusDefaultBrowser)
         Browser.localeOverride.set(components.store.state.locale?.displayName ?: "none")
         val shortcutsOnHomeNumber = components.topSitesStorage.getTopSites(

diff --git a/app/src/main/java/org/mozilla/gecko/search/SearchWidgetProvider.kt b/app/src/main/java/org/mozilla/gecko/search/SearchWidgetProvider.kt
@@ -14,6 +14,7 @@ import mozilla.components.support.utils.PendingIntentUtils
 import org.mozilla.focus.R
 import org.mozilla.focus.activity.IntentReceiverActivity
 import org.mozilla.focus.ext.components
+import org.mozilla.focus.searchwidget.VoiceSearchActivity
 
 class SearchWidgetProvider : AppSearchWidgetProvider() {
 
@@ -42,6 +43,7 @@ class SearchWidgetProvider : AppSearchWidgetProvider() {
                 val createTextSearchIntentFlags = PendingIntentUtils.defaultFlags or
                     PendingIntent.FLAG_UPDATE_CURRENT
                 intent.flags = Intent.FLAG_ACTIVITY_NEW_TASK or Intent.FLAG_ACTIVITY_CLEAR_TASK
+                intent.putExtra(IntentReceiverActivity.SEARCH_WIDGET, true)
                 PendingIntent.getActivity(
                     context,
                     REQUEST_CODE_NEW_TAB, intent, createTextSearchIntentFlags
@@ -54,7 +56,7 @@ class SearchWidgetProvider : AppSearchWidgetProvider() {
     }
 
     override fun voiceSearchActivity(): Class<out BaseVoiceSearchActivity> {
-        return BaseVoiceSearchActivity::class.java
+        return VoiceSearchActivity::class.java
     }
 
     companion object {