1833245 - Allow page clicks for cached pages to be correctly recorded

Status: RESOLVED FIXED

(Firefox :: Search, task, P2)

Description

[James Teow [:jteow]]

SearchSERPTelemetry listens to activity on http-on-examine-response

AFAIK, this doesn't include cached response. For that, we'd need to also listen to http-on-examine-cached-response or use http-on-examine-merged-response.

Our current solution works for the vast majority of situations, as partners often use tracked links on their SERP first make a request to a non-cached resource before being re-directed to the search result link.

However for partners that don't track non-ads, this could result in a situation where we don't record an event, specifically clicking on a non-ads click after its been cached.

Example STR:

1. Use a SERP that doesn't track non ad links (e.g. DDG, Ecosia)
2. Search for a page that is cached (e.g. an article on MDN)
3. Visit the page.
4. Go back to the SERP.
5. Visit the page again.

Visiting the page a second time from the SERP will record an abandonment and we won't record a page click.

Comment 1

[James Teow [:jteow]]

Attachment: Bug 1833245 - Enable cached page loads to be monitored in SearchSERPTelemetry - r?standard8,scunnane

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e73b9e917dd1
Enable cached page loads to be monitored in SearchSERPTelemetry - r=scunnane

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/e73b9e917dd1