Add UI Telemetry for search history items

RESOLVED FIXED in Firefox 43

Status

()

Firefox for Android
Awesomescreen
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mfinkle, Assigned: ally)

Tracking

unspecified
Firefox 43
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox43 fixed)

Details

Attachments

(1 attachment)

We currently collect UI Telemetry on the search suggestions returned from the default search engine. We tag those with the Extra "engine.#" where "#" is the position:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/home/SearchEngineRow.java#95

I think we are currently mixing the search history into the suggestions:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/home/SearchEngineRow.java#194

We don't differentiate between engine suggestion of history suggestion. I'd like to see us use "history.#" for the search history, where "#" is the 0 based index of the history items.
If we want to keep things relatively simple, here are some less-than-awesome suggestions:
1. Offset the Tag of history items by 1000 here [1]. Use >= 1000 here [2] as a hint we have a history item.
2. Use the visible history image as a hint that we have a history item.

[1] http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/home/SearchEngineRow.java#195 (isUserSavedSearch)
[2] http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/home/SearchEngineRow.java#95
(Assignee)

Comment 2

3 years ago
That is correct, they are currently mixed, and the position is relative to the length of the list.
(Assignee)

Updated

3 years ago
Blocks: 1086952
(Assignee)

Comment 3

3 years ago
Created attachment 8660999 [details] [diff] [review]
SavedSearchUITelemetry

Your name is at the top of the suggested reviewers list by the way.

I agree those are less than awesome suggestions. :) 

This patch makes use of the Tag we already create on every SuggestionView to encode the data we'd like to pass to telemetry. Previously it was just the global index. Now it contains the type (history or engine) and the relative 0 based index for that type. 

Note for those playing along at home: the first 'suggestion' which is just a copy of what the user has in the urlbar, has a separate/special tag in UITelemetry, so is not affected by this patch and not listed in the output below.

Built on top of Bug 1200319, going from left to right in en-us, the tags in UItelemetry are

phone
31613-31613/org.mozilla.fennec_mozilla W/anaaktge﹕ onclickhandler tag = engine.0
31613-31613/org.mozilla.fennec_mozilla W/anaaktge﹕ onclickhandler tag = engine.1
31613-31613/org.mozilla.fennec_mozilla W/anaaktge﹕ onclickhandler tag = history.0
31613-31613/org.mozilla.fennec_mozilla W/anaaktge﹕ onclickhandler tag = history.1

tablet
6685-6685/org.mozilla.fennec_mozilla W/anaaktge﹕ onclickhandler tag = engine.0
6685-6685/org.mozilla.fennec_mozilla W/anaaktge﹕ onclickhandler tag = engine.1
6685-6685/org.mozilla.fennec_mozilla W/anaaktge﹕ onclickhandler tag = engine.2
6685-6685/org.mozilla.fennec_mozilla W/anaaktge﹕ onclickhandler tag = history.0
6685-6685/org.mozilla.fennec_mozilla W/anaaktge﹕ onclickhandler tag = history.1
6685-6685/org.mozilla.fennec_mozilla W/anaaktge﹕ onclickhandler tag = history.2
6685-6685/org.mozilla.fennec_mozilla W/anaaktge﹕ onclickhandler tag = history.3
Attachment #8660999 - Flags: review?(mark.finkle)
Comment on attachment 8660999 [details] [diff] [review]
SavedSearchUITelemetry


>diff --git a/mobile/android/base/home/SearchEngineRow.java b/mobile/android/base/home/SearchEngineRow.java

>+    private void bindSuggestionView(String suggestion, boolean animate, int recycledSuggestionCount, Integer previousSuggestionChildIndex, boolean isUserSavedSearch, String telmetryTag){

telmetryTag -> telemetryTag

r+ with the fixup
Attachment #8660999 - Flags: review?(mark.finkle) → review+
(Assignee)

Comment 5

3 years ago
The lowest grade I ever got in high school _was_ in spelling...
https://hg.mozilla.org/mozilla-central/rev/2a5e16abd240
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox43: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
You need to log in before you can comment on or make changes to this bug.