Closed Bug 1398270 Opened 7 years ago Closed 4 years ago

Unknown source for one-off search: header error when clicking on search header

Categories

(Firefox :: Search, defect, P2)

defect

Tracking

()

VERIFIED FIXED
86 Branch
Tracking Status
firefox86 --- verified

People

(Reporter: mkaply, Assigned: standard8)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fixed by bug 1662553][fxsearch][search-telemetry-backlog])

After typing in a search, you can show the dropdown and then click on the header "Yahoo search" or "Google search" and a search will be executed. You'll see this error in the console though: Unknown source for one-off search: header BrowserUsageTelemetry.jsm:405 This is actually a known source for one-off, so we should be reporting it correctly.
Sounds like one just needs to add the source to KNOWN_ONEOFF_SOURCES in BrowserUsageTelemetry near http://searchfox.org/mozilla-central/source/browser/modules/BrowserUsageTelemetry.jsm#50 Alessio, is this so?
Flags: needinfo?(alessio.placitelli)
Component: Telemetry → Search
Flags: needinfo?(alessio.placitelli) → needinfo?(mak77)
Product: Toolkit → Firefox
Well, it's not exactly a one-off, nor it's a normal search. It's wrong that we don't account for it, but it may also be wrong to count it as a one-off search. I must also note this doesn't happen for the urlbar that is currently our primary target, though the urlbar shows the default engine, thus we could consider this header the same thing? It's also only reachable through mouse click, that makes it a little bit of an edge-case. This is something to be evaluated directly by who manages/analyzes the collected data. Dave?
Flags: needinfo?(mak77) → needinfo?(dzeber)
I'm surprised the search bar header is clickable. I would have expected it to match the awesomebar header which isn't. Adding Philipp for UX guidance here.
Flags: needinfo?(philipp)
Priority: -- → P2
Sorry for the delay. I don't think this should be counted as a one-off. I understand one-offs to mean that the user consciously opted to use a non-default engine. This is just a different way of getting to a default-engine searchbar search. That said, the only place where this makes a difference is in UITelemetry (and possibly the browser.engagement.navigation scalars), which is not commonly used at the moment. The oneoff sources are not recorded in standard Telemetry SEARCH_COUNTS. If we were counting these header searches, they would show up as `<engine>.searchbar` - the same as any other searchbar search using the default engine. (One-off searches are detected in standard Telemetry as those using a non-default engine). I agree that it should be fixed. However, I expect this is an edge case, even more so with the searchbar going away for most people in 57.
Flags: needinfo?(dzeber)
(In reply to Panos Astithas [:past] (please ni?) from comment #3) > I'm surprised the search bar header is clickable. I would have expected it > to match the awesomebar header which isn't. Adding Philipp for UX guidance > here. There may be some confusion on the word "header" here. In the search bar we are talking about the first line that's at the top of the panel, above suggestions, and that shows what the default engine currently is. We made it clickable in bug 1110235 per Philipp's request. I think in the awesomebar when you say "header" you are talking about the line above the one-off buttons that says "Search for <query> with:".
Flags: needinfo?(philipp)
This bug causes noise when running the browser/components/search/test/browser_oneOffHeader.js test.
You also get the same error if you use paste and search: Unknown source for one-off search: paste The problem here seems to be there are things that aren't one off searches (paste, header) that we are counting as such here: https://searchfox.org/mozilla-central/source/browser/components/search/content/search.xml#424 We need to do something different because a) we're producing an error and b) we're not recording this data in telemetry. We just need to decide what the right thing is and just do it.
Whiteboard: [fxsearch] → [fxsearch][search-telemetry-backlog]
Severity: normal → S4
Type: enhancement → defect

Bug 1662553 is going to be dropping this error message.

Depends on: 1662553

Fixed by bug 1662553.

Assignee: nobody → standard8
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Whiteboard: [fxsearch][search-telemetry-backlog] → [fixed by bug 1662553][fxsearch][search-telemetry-backlog]
Target Milestone: --- → 86 Branch

Hi,
I verified that I could reproduce this bug with 85.0a1 (2020-12-02) (64-bit) and then I verified that this is fixed on 86.0a1 (2021-01-05) (64-bit) on w10.

Regards, Flor.

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