Closed Bug 1695747 Opened 3 years ago Closed 3 years ago

Remove sponsored top sites from address bar results

Categories

(Firefox :: Address Bar, task, P1)

Firefox 87
Desktop
All
task
Points:
2

Tracking

()

VERIFIED FIXED
88 Branch
Iteration:
88.1 - Feb 22 - Mar 7
Tracking Status
firefox87 --- verified
firefox88 --- verified

People

(Reporter: jimthomas, Assigned: dao)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Sponsored top sites should not appear by default in the address bar. Previously this work was done for an experiment but should not be included in the default experience.

  • Remove Sponsored Top Sites from address bar
  • Organic Top Sites Should show in address bar
  • Sponsored Top Sites should remain on about:newtab
Assignee: nobody → dao+bmo
Severity: -- → N/A
Status: NEW → ASSIGNED
Type: defect → task
Iteration: --- → 88.1 - Feb 22 - Mar 7
Points: --- → 2
Priority: -- → P1
See Also: → 1694620
Blocks: 1690584
Component: Top Sites → Address Bar
See Also: → 1608418
See Also: → 1628025
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/73339fe01d1b
Remove sponsored top sites from address bar results. r=mikedeboer

Backed out changeset 73339fe01d1b (Bug 1695747) for causing bc failures in browser_urlbar_telemetry_sponsored_topsites.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/c3d29f1a05dc18950522c4f11e1f54479f0c73b6
Push with failures, failure log.

Flags: needinfo?(dao+bmo)
Flags: needinfo?(dao+bmo)
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2dc290926288
Remove sponsored top sites from address bar results. r=mikedeboer
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch

Comment on attachment 9206246 [details]
Bug 1695747 - Remove sponsored top sites from address bar results. r=mikedeboer

Beta/Release Uplift Approval Request

  • User impact if declined: There was some confusion with sponsored sites in the address bar and users not understanding how to disable them. This is a high priority as per https://jira.mozilla.com/browse/FIDE-406
  • 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: I'm talking to QA to figure this out
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Patch is quite simple
  • String changes made/needed:
Attachment #9206246 - Flags: approval-mozilla-beta?
Flags: qe-verify+
QA Contact: adrian.florinescu

Comment on attachment 9206246 [details]
Bug 1695747 - Remove sponsored top sites from address bar results. r=mikedeboer

approved for 87.0b6

Attachment #9206246 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Verified as fixed on Linux Mint 20, Windows 10 and Mac 10.14.6 using Firefox Nightly 88.0a1 2021-03-03. (aided by the browser.topsites.experiment.ebay-2020-1 pref. in order to get sponsored tiles)

QA Whiteboard: [qa-triaged]

Verified the fix on 87.0b6 2021-03-04, using:

Preconditions: (to enable sponsored topsites)
   user_pref("browser.search.region","US");
   user_pref("browser.topsites.experiment.ebay-2020-1", true);
Os coverage:
   Ubuntu 18.04
   Mac 10.14.6
   Windows 10

Aditionally, we also verified that getting frecency counts on the site will add the topsite in cause as addressbar topsite if the sponsored would get dissabled.

e.g.:

  1. amazon.com -> sponsored
  2. generate amazon.com brwosing history
  3. Switch the sponsored topsites off.

AR/ER:

  1. amzon.com as sponsored topsite not present in address bar.
  2. ammazon.com as topsite present in address bar.
Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-triaged]
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: