1664203 - "Send attribution request" telemetry and ping do not work after the first restart

Status: VERIFIED FIXED

(Firefox :: Top Sites, defect, P1)

Description

[Adrian Florinescu [:aflorinescu] [PTO until Aug 26]]]

[Sugested Severity:]
S2

[Description:]
With "Send attribution request" set for a topsite, a ping anda telemetry event should be performed for each action.
The above works until first restart, afterwhich no ping and no telemetry is logged.

[Environment:]
Windows 10, Ubuntu 20, Mac 10.13.6
81.0b8

[Steps:]

1. From Remote Settings administration, add a general topsite.
2. From the Remote Settings configuration, set the above topsite to "Send attribution request to Mozilla server"
3. Publish the changes and set Firefox to connect to the respective collection.
4. Set a Firefox profile to connect to the configuration while switching the browser.topsites.useRemoteSetting to true.
5. Start-up Firefox with the above profile configuration.
6. Open a new tab and select from the address bar the topsite previously created.
7. Open a new tab and select the previously selected topsite from the Topsites section in the about:newtab page.
8. Restart the browser.
9. Open a new tab and select from the address bar the topsite previously created.
10. Open a new tab and select the previously selected topsite from the Topsites section in the about:newtab page.

[Actual Result:]

6,7. : In browser console, enabling XHR, a ping to https://topsites.mozilla.io/cid/amzn_2020_a1 (2x)
6,7,.: Opening about:telemetry#events-tab in a new tab will show:

partner_link interaction click {"partner": "google", "source": "newtab"}
partner_link interaction attribution success {"partner": "google", "source": "newtab"}
partner_link interaction click {"partner": "google", "source": "urlbar"}
partner_link interaction attribution success {"partner": "google", "source": "urlbar"}

After restart, the steps 9 and 10 do not log any ping or telemetry

[Expected Result:]

6,7,9,10. : In browser console, enabling XHR, a ping to https://topsites.mozilla.io/cid/amzn_2020_a1 (4x)
6,7,9,10.: Opening about:telemetry#events-tab in a new tab will show (2x):

partner_link interaction click {"partner": "google", "source": "newtab"}
partner_link interaction attribution success {"partner": "google", "source": "newtab"}
partner_link interaction click {"partner": "google", "source": "urlbar"}
partner_link interaction attribution success {"partner": "google", "source": "urlbar"}

Comment 1

[Dão Gottwald [:dao]]

So the problem here is that the google tile becomes a frecency tile after restart, for which we don't send attribution requests. This is somewhat expected when the exact URL of a frecency tile is different from the default site URL. However when the frecency tile has the exact same URL as the original default tile, I think we'd need the request. I'm not sure that we'll run into this with Amazon (i.e. whether https://www.amazon.com/?tag=partnerid&ref=foobar would end up becoming a frecency tile).

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/25ad850e09ec
Send attribution request work for default tiles that became frecency tiles. r=mikedeboer

Comment 3

[Dão Gottwald [:dao]]

Attachment: Bug 1664203 - Send attribution request for default tiles that became frecency tiles. r=mikedeboer

Comment 4

[Ryan VanderMeulen [:RyanVM]]

Does this need to be uplifted to 81? Please nominate for release approval ASAP if so.

Comment 5

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/25ad850e09ec

Comment 6

[Dão Gottwald [:dao]]

Comment on attachment 9175781 [details]
Bug 1664203 - Send attribution request for default tiles that became frecency tiles. r=mikedeboer

Beta/Release Uplift Approval Request

• User impact if declined: See comment 0
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: See comment 0
• List of other uplifts needed: Bug 1664502
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Straightforward fix. Top sites from remote settings are also still disabled by default -- we intend to roll out the feature after release to minimize risk.
• String changes made/needed:

Comment 7

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9175781 [details]
Bug 1664203 - Send attribution request for default tiles that became frecency tiles. r=mikedeboer

Approved for 81.0rc2.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-release/rev/ad30a5441d7f

Comment 9

[Adrian Florinescu [:aflorinescu] [PTO until Aug 26]]]

Verified the issue as fixed with 82.0a1 2020-09-17 and 81RC2 using Mac 10.13.6 and Windows 10.

Comment 10

[Adrian Florinescu [:aflorinescu] [PTO until Aug 26]]]

Seems that the https://www.example.com/#%YYYYMMDDHH% has a special code-path and the timestamp scenario was not covered properly as part of comment 9 scenario.
Leaving this bug verified for generic topsites and spinning bug 1665757 for timestamp type sites.