Closed Bug 1505456 Opened 7 years ago Closed 7 years ago

Move recordSearchURLTelemetry to its own module

Categories

(Firefox :: Search, enhancement, P1)

Product:
Firefox
Component:
Search
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 65
Tracking Flags:
Tracking Status
firefox65 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

(Whiteboard: [fxsearch])

Attachments

(2 files)

Bug 1505456 - Move browser tests into a sub-directory for browser/components/search/test.
47 bytes, text/x-phabricator-request
Details | Review
Bug 1505456 - Move recordSearchURLTelemetry to its own module.
47 bytes, text/x-phabricator-request
Details | Review
As part of bug 1505411, I think it makes sense to handle everything related to telemetry for searches in a dedicated SearchTelemetry module. There's nothing dependant on the search service internals here, and having a dedicated module means we know exactly where to go for it, and it can be tested more easily.
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3589ac87b808 Move browser tests into a sub-directory for browser/components/search/test. r=adw https://hg.mozilla.org/integration/autoland/rev/4056c05b187d Move recordSearchURLTelemetry to its own module. r=adw
https://hg.mozilla.org/mozilla-central/rev/3589ac87b808 https://hg.mozilla.org/mozilla-central/rev/4056c05b187d
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox65: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: