Closed Bug 1847855 Opened 2 years ago Closed 2 years ago

Add additional fields to the Glean urlbar events

Categories

(Firefox :: Address Bar, task, P1)

task

Tracking

()

VERIFIED FIXED
118 Branch
Tracking Status
firefox118 --- verified
firefox119 --- verified

People

(Reporter: daisuke, Assigned: daisuke)

References

()

Details

Attachments

(3 files, 2 obsolete files)

No description provided.
Attached file data-review-request-for-bug1847855.md (obsolete) —
Attachment #9348314 - Flags: data-review?(mmccorquodale)
Attached file data-review-request-for-bug1847855.md (obsolete) —

Please let me update the data-review file because some telemetry names were changed.

Attachment #9348314 - Attachment is obsolete: true
Attachment #9348314 - Flags: data-review?(mmccorquodale)
Attachment #9348350 - Flags: data-review?(mmccorquodale)

Sorry, please let me update the content of data-review again since the one variable name was changed.

Attachment #9348350 - Attachment is obsolete: true
Attachment #9348350 - Flags: data-review?(mmccorquodale)
Attachment #9348504 - Flags: data-review?(mmccorquodale)
Attachment #9348108 - Attachment description: Bug 1847855: Take telemetry of default search engine and selected index → Bug 1847855: Take telemetry of default search engine and selected position
Severity: -- → N/A
Priority: -- → P1
Attachment #9348504 - Flags: data-review?(mmccorquodale) → data-review?(tlong)

Comment on attachment 9348504 [details]
data-review-request-for-bug1847855.md

Permanent collections require an individual contributor to take on the responsibility of checking on this data from time to time to ensure it is necessary and useful.

This is noted in the instructions for question 7 in the data-collection template:

  1. How long will this data be collected? Choose one of the following:
  • This is scoped to a time-limited experiment/project until date MM-DD-YYYY.

  • I want this data to be collected for 6 months initially (potentially renewable).

  • I want to permanently monitor this data. (put someone’s name here)

Can you please state who that individual contributor would be and I will be able to complete my review, thanks!

Flags: needinfo?(scunnane)

Comment on attachment 9348504 [details]
data-review-request-for-bug1847855.md

  1. Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?
    Yes, this will be documented in the Glean dictionary.

  2. Is there a control mechanism that allows the user to turn the data collection on and off?
    Yes, users can opt out of telemetry collection.

  3. If the request is for permanent data collection, is there someone who will monitor the data over time?
    This will be monitored by the search team.

  4. Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
    Category 2, Interaction Data.

  5. Is the data collection request for default-on or default-off?
    Default on.

  6. Does the instrumentation include the addition of any new identifiers?
    No new identifiers.

  7. Is the data collection covered by the existing Firefox privacy notice?
    Yes.

  8. Does the data collection use a third-party collection tool?
    No.


data-review +

Attachment #9348504 - Flags: data-review?(tlong) → data-review+

The individual contributors who will monitor this over time will be Drew Willcox And Dave Zeber

Flags: needinfo?(scunnane)
Pushed by dwillcoxon@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/91c127fef1ca Take telemetry of the prefs that enable/disable part of QuickSuggest features r=adw https://hg.mozilla.org/integration/autoland/rev/200aa08042f1 Take telemetry of default search engine and selected position r=adw
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch
Regressions: 1849726

Some telemetry names have been changed in Bug 1849726, due to technical restrictions:
pref_quicksuggest_data_collection => pref_suggest_data_collection
pref_quicksuggest_nonsponsored => pref_suggest_nonsponsored
pref_quicksuggest_sponsored => pref_suggest_sponsored

Is there any documentation in regards to these changes that QA can use for reference?

Flags: needinfo?(scunnane)
Flags: needinfo?(daisuke)
Flags: qe-verify+

Adrian, Daisuke is the better person to ask about the docs.

Flags: needinfo?(scunnane)

Hi Adrian! I'm sorry for my delay. I explain here.

Telemetry for preference:

  • pref_suggest_data_collection
    • Is recorded the browser.urlbar.quicksuggest.dataCollection.enabled value at initializing and when the pref is changed.
  • pref_suggest_nonsponsored
    • Is recorded the browser.urlbar.suggest.quicksuggest.nonsponsored value at initializing and when the pref is changed.
  • pref_suggest_sponsored
    • Is recorded the browser.urlbar.suggest.quicksuggest.sponsored value at initializing and when the pref is changed.

Additional value to abandonment, engagement, and impression event:

  • search_engine_default_id
    • The telemetry id of the search engine. Reflects search.engine.default.engine_id.

Additional value to engagement event:

  • selected_position
    • The 1-based index of the result the user selected. If user searched without selection, 0 will be recorded.
Flags: needinfo?(daisuke)

Thanks for the info Daisuke.
We've tested the above with the following questions:

Telemetry for preference:

  • pref_suggest_data_collection
    • Is recorded the browser.urlbar.quicksuggest.dataCollection.enabled value at initializing and when the pref is changed.

Seems to be recorded into all events, not only at initialization/pref change. recorded glean_debug

  • pref_suggest_nonsponsored
    • Is recorded the browser.urlbar.suggest.quicksuggest.nonsponsored value at initializing and when the pref is changed.

Seems to be recorded into all events, not only at initialization/pref change. recorded glean_debug

  • pref_suggest_sponsored
    • Is recorded the browser.urlbar.suggest.quicksuggest.sponsored value at initializing and when the pref is changed.

Seems to be recorded into all events, not only at initialization/pref change. recorded glean_debug

Additionally, we also see urlbar.pref_suggest_topsites, I'd guess this enters the same category as above.

Additional value to engagement event:

  • selected_position
    • The 1-based index of the result the user selected. If user searched without selection, 0 will be recorded.

I'm not sure how the value 0 can be triggered here: how can you search without a selection?

Flags: needinfo?(daisuke)

Hello!

(In reply to Adrian Florinescu [:aflorinescu] from comment #15)

We've tested the above with the following questions:

Telemetry for preference:

  • pref_suggest_data_collection
    • Is recorded the browser.urlbar.quicksuggest.dataCollection.enabled value at initializing and when the pref is changed.

Seems to be recorded into all events, not only at initialization/pref change. recorded glean_debug

  • pref_suggest_nonsponsored
    • Is recorded the browser.urlbar.suggest.quicksuggest.nonsponsored value at initializing and when the pref is changed.

Seems to be recorded into all events, not only at initialization/pref change. recorded glean_debug

  • pref_suggest_sponsored
    • Is recorded the browser.urlbar.suggest.quicksuggest.sponsored value at initializing and when the pref is changed.

Seems to be recorded into all events, not only at initialization/pref change. recorded glean_debug

Additionally, we also see urlbar.pref_suggest_topsites, I'd guess this enters the same category as above.

Ah, yes, urlbar.pref_suggest_topsites is handled as same as other pref changes above.

And, regarding the inclusion of metrics in events.
I don't know much about this area, but it seems that when data is pinged, metrics is also included.
https://mozilla.github.io/glean/book/user/pings/index.html#payload-structure

Additional value to engagement event:

  • selected_position
    • The 1-based index of the result the user selected. If user searched without selection, 0 will be recorded.

I'm not sure how the value 0 can be triggered here: how can you search without a selection?

Yes, if user opens a page without showing urlbar result, 0 is recorded.
For example, if user does paste and go on the urlbar, the value will be 0.
STR:

  1. Copy http://example.com to clipboard.
  2. Open context menu on urlbar
  3. Select Paste and Go
Flags: needinfo?(daisuke)

Thanks Daisuke for the clarification. Indeed that's how you generate the 0 value. And with the additional explanation, this bug is verified on Windows 10, Mac 13, Ubuntu 22 with Firefox 118.0b5, Firefox 119.0a1 (2023-09-07).

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: