Skip to content

Commit

Permalink
Remove search suggestions in private telemetry
Browse files Browse the repository at this point in the history
  • Loading branch information
eliserichards authored and mergify[bot] committed Aug 31, 2021
1 parent 19d8f58 commit c962601
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 24 deletions.
18 changes: 0 additions & 18 deletions app/metrics.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3366,24 +3366,6 @@ logins:
- android-probes@mozilla.com
expires: "2022-02-01"

search_suggestions:
enable_in_private:
type: event
description: |
A user enabled receiving search suggestions in private sessions
bugs:
- https://github.com/mozilla-mobile/fenix/issues/6070
data_reviews:
- https://github.com/mozilla-mobile/fenix/pull/6746
- https://github.com/mozilla-mobile/fenix/pull/15713#issuecomment-703972068
- https://github.com/mozilla-mobile/fenix/pull/19924#issuecomment-861423789
data_sensitivity:
- technical
- interaction
notification_emails:
- android-probes@mozilla.com
expires: "2021-08-01"

voice_search:
tapped:
type: event
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,6 @@ sealed class Event {
object EditLogin : Event()
object EditLoginSave : Event()
object ViewLoginPassword : Event()
object PrivateBrowsingShowSearchSuggestions : Event()
object WhatsNewTapped : Event()
object PocketTopSiteClicked : Event()
object PocketTopSiteRemoved : Event()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ import org.mozilla.fenix.GleanMetrics.ReaderMode
import org.mozilla.fenix.GleanMetrics.RecentBookmarks
import org.mozilla.fenix.GleanMetrics.RecentTabs
import org.mozilla.fenix.GleanMetrics.SearchShortcuts
import org.mozilla.fenix.GleanMetrics.SearchSuggestions
import org.mozilla.fenix.GleanMetrics.SearchWidget
import org.mozilla.fenix.GleanMetrics.SetDefaultNewtabExperiment
import org.mozilla.fenix.GleanMetrics.SetDefaultSettingExperiment
Expand Down Expand Up @@ -443,9 +442,6 @@ private val Event.wrapper: EventWrapper<*>?
is Event.EditLoginSave -> EventWrapper<NoExtraKeys>(
{ Logins.saveEditedLogin.record(it) }
)
is Event.PrivateBrowsingShowSearchSuggestions -> EventWrapper<NoExtraKeys>(
{ SearchSuggestions.enableInPrivate.record(it) }
)
is Event.ToolbarPositionChanged -> EventWrapper(
{ ToolbarSettings.changedPosition.record(it) },
{ ToolbarSettings.changedPositionKeys.valueOf(it) }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -318,7 +318,6 @@ class SearchDialogFragment : AppCompatDialogFragment(), UserInteractionHandler {
}
store.dispatch(SearchFragmentAction.SetShowSearchSuggestions(true))
store.dispatch(SearchFragmentAction.AllowSearchSuggestionsInPrivateModePrompt(false))
requireComponents.analytics.metrics.track(Event.PrivateBrowsingShowSearchSuggestions)
}

searchSuggestionHintBinding.dismiss.setOnClickListener {
Expand Down

0 comments on commit c962601

Please sign in to comment.