For #6557 - added metric for in_content
parent
f10e202a1e
commit
f09bc4566c
|
@ -2030,6 +2030,20 @@ browser.search:
|
||||||
notification_emails:
|
notification_emails:
|
||||||
- fenix-core@mozilla.com
|
- fenix-core@mozilla.com
|
||||||
expires: "2020-09-01"
|
expires: "2020-09-01"
|
||||||
|
in_content:
|
||||||
|
type: labeled_counter
|
||||||
|
description: >
|
||||||
|
Records the type of interaction a user has on SERP pages.
|
||||||
|
send_in_pings:
|
||||||
|
- baseline
|
||||||
|
- metrics
|
||||||
|
bugs:
|
||||||
|
- https://github.com/mozilla-mobile/fenix/issues/6557
|
||||||
|
data_reviews:
|
||||||
|
- https://github.com/mozilla-mobile/fenix/pull/10167
|
||||||
|
notification_emails:
|
||||||
|
- fenix-core@mozilla.com
|
||||||
|
expires: "2020-09-01"
|
||||||
|
|
||||||
addons:
|
addons:
|
||||||
open_addons_in_settings:
|
open_addons_in_settings:
|
||||||
|
|
|
@ -117,6 +117,11 @@ private val Event.wrapper: EventWrapper<*>?
|
||||||
BrowserSearch.adClicks[label].add(1)
|
BrowserSearch.adClicks[label].add(1)
|
||||||
}
|
}
|
||||||
)
|
)
|
||||||
|
is Event.SearchInContent -> EventWrapper<NoExtraKeys>(
|
||||||
|
{
|
||||||
|
BrowserSearch.inContent[label].add(1)
|
||||||
|
}
|
||||||
|
)
|
||||||
is Event.SearchShortcutSelected -> EventWrapper(
|
is Event.SearchShortcutSelected -> EventWrapper(
|
||||||
{ SearchShortcuts.selected.record(it) },
|
{ SearchShortcuts.selected.record(it) },
|
||||||
{ SearchShortcuts.selectedKeys.valueOf(it) }
|
{ SearchShortcuts.selectedKeys.valueOf(it) }
|
||||||
|
|
|
@ -375,6 +375,11 @@ sealed class Event {
|
||||||
get() = providerName
|
get() = providerName
|
||||||
}
|
}
|
||||||
|
|
||||||
|
data class SearchInContent(val keyName: String) : Event() {
|
||||||
|
val label: String
|
||||||
|
get() = keyName
|
||||||
|
}
|
||||||
|
|
||||||
class ContextMenuItemTapped private constructor(val item: String) : Event() {
|
class ContextMenuItemTapped private constructor(val item: String) : Event() {
|
||||||
override val extras: Map<ContextMenu.itemTappedKeys, String>?
|
override val extras: Map<ContextMenu.itemTappedKeys, String>?
|
||||||
get() = mapOf(ContextMenu.itemTappedKeys.named to item)
|
get() = mapOf(ContextMenu.itemTappedKeys.named to item)
|
||||||
|
|
File diff suppressed because one or more lines are too long
Loading…
Reference in New Issue