Closed
Bug 1286594
Opened 8 years ago
Closed 7 years ago
Search performed in Activity Stream are not being counted
Categories
(Firefox :: Activity Streams: General, defect, P2)
Firefox
Activity Streams: General
Tracking
()
RESOLVED
WONTFIX
Tracking | Status | |
---|---|---|
firefox50 | --- | affected |
People
(Reporter: Dexter, Unassigned)
References
Details
(Whiteboard: [fxsearch])
Opening a new tab with the Activity Stream addon installed shows a page with a search field. Searching through that field isn't being counted in the SEARCH_COUNTS keyed histogram.
STR:
- Open about:telemetry and inspect the SEARCH_COUNTS keyed histogram
- Open a new tab
- Perform a search in the search box within the new tab page.
- Refresh the about:telemetry page.
- SEARCH_COUNT doesn't get updated with the new search.
Comment 1•8 years ago
|
||
This is how we currently record searches in core Firefox code:
https://dxr.mozilla.org/mozilla-central/search?q=BrowserSearch.record&redirect=false
Reporter | ||
Comment 2•8 years ago
|
||
Panos, I'm just ni? you as a heads-up to make sure you're aware of this bug (this prevents us from counting the related data for bug 1303333 as well).
Flags: needinfo?(past)
Comment 3•8 years ago
|
||
Thanks for the heads up. Activity Stream is a Test Pilot add-on and I don't know how telemetry works there, but it seems to me that this is where we would want to instrument the code:
https://github.com/mozilla/activity-stream/blob/2d22061c549cdd27bb75ceb2fdcd931d92f7babd/addon/SearchProvider.js#L187
Flags: needinfo?(past)
Updated•8 years ago
|
Priority: -- → P2
Updated•8 years ago
|
Whiteboard: [fxsearch]
Comment 4•7 years ago
|
||
I think Tim mentioned in a firefox desktop meeting a while ago that the version of test pilot in the tree is taking advantage of the existing search box component and therefore searches through that Search Access Point are being tracked as about:newtab searches. Tim can you confirm?
In that case, do we still want to track searches for the test pilot population or is the number small enough that we don't care?
Also, will the test pilot version ever reuse the in-tree search box component to make this issue disappear entirely?
Flags: needinfo?(tspurway)
Comment 5•7 years ago
|
||
Although we aren't tracking searches in the Test Pilot add-on with regular telemetry, we *are* in the release version of Activity Stream that has landed in Nightly. If you enable Activity Stream in Nightly (pref on browser.newtabpage.activity-stream.enabled), and redo the above test, it passses.
Since the Test Pilot add-on has 'graduated' and will soon go into a code freeze, I don't think we will be updating it to properly record search telemetry.
Flags: needinfo?(tspurway)
Comment 6•7 years ago
|
||
Thanks Tim. Javaun, do you agree that it makes sense to wontfix this bug for AS in test pilot?
Flags: needinfo?(jmoradi)
Comment 7•7 years ago
|
||
Agree, we can skip this in the test pilot addon as long as the version landing in Fx had the probes
Flags: needinfo?(jmoradi)
You need to log in
before you can comment on or make changes to this bug.
Description
•