Closed Bug 1847787 Opened 3 years ago Closed 3 years ago

Pocket newtab sponsored topsite enabled by default

Categories

(Firefox :: New Tab Page, enhancement)

enhancement

Tracking

()

VERIFIED FIXED
118 Branch
Tracking Status
firefox117 + verified
firefox118 --- verified

People

(Reporter: thecount, Assigned: thecount)

References

(Depends on 1 open bug)

Details

Attachments

(1 file)

No description provided.
Summary: Pocket newtab sponsored topsite enabled by defaulkt → Pocket newtab sponsored topsite enabled by default
Assignee: nobody → sdowne
Status: NEW → ASSIGNED
Pushed by sdowne@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7544c8f9eea7 Pocket newtab sponsored topsites enabled by default r=nanj

Backed out for causing marionette crashes with @ mozilla::net::nsHttpChannel::OnStartRequest

Flags: needinfo?(sdowne)
Pushed by sdowne@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/181edf360c56 Pocket newtab sponsored topsites enabled by default r=nanj,webdriver-reviewers,whimboo
Pushed by sdowne@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5af04f9a95e0 Pocket newtab sponsored topsites enabled by default r=nanj,webdriver-reviewers,whimboo
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch

Comment on attachment 9347974 [details]
Bug 1847787 - Pocket newtab sponsored topsites enabled by default

Beta/Release Uplift Approval Request

  • User impact if declined: Revenue impact
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: We don't really have sponsored Pocket content for this currently, so even if it's turned on, you won't see something.

So let's just do regression testing on topsites, specifically check that existing sponsored topsites still show.

  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): It's a basic code change, and a pref flip. The rest of the changes were to tests.

It has also been turned on for various experiments and tested that way.

  • String changes made/needed: None
  • Is Android affected?: No
Flags: needinfo?(sdowne)
Attachment #9347974 - Flags: approval-mozilla-beta?
Flags: qe-verify+

[Tracking Requested - why for this release]:

Comment on attachment 9347974 [details]
Bug 1847787 - Pocket newtab sponsored topsites enabled by default

Approved for 117.0b9.

Attachment #9347974 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
QA Whiteboard: [qa-triaged]

We verified this issue on the latest Firefox Nightly 118.0a1 (Build ID: 20230816215210) on Windows 10 x64, macOS 12.6.1, and Ubuntu 20.04 x64. Since the Pocket tile is not available for testing, we performed regression testing on the existing sponsored and non-sponsored topsites. The following scenarios were covered across all 3 platforms:

  1. Verified that the Sponsored Tiles are displayed on certain regions (US, FR, ES, and DE).
  2. Verified that the Sponsored Tiles redirect to the correct page.
  3. Verified that the impression/click pings are registered in the Browser Console.
  4. Verified that the Sponsored Tiles cannot be moved to another position.
  5. Verified each option from the Sponsored Tile menu (...).
  6. Verified that the Sponsored Tiles can be dismissed.
  7. Verified that the Sponsored Tiles are displayed if a New Window is opened.
  8. Verified that if moving Non-Sponsored Tiles nothing is broken in the New Tab’s Page UI.

We verified this issue on the latest Firefox Beta 117.0b9 (Build ID: 20230817175856) on Windows 10 x64, macOS 13.1, and Linux Mint 21.1 Cinnamon 5.6.5. We performed regression testing on the existing sponsored and non-sponsored topsites. The following scenarios were covered across all 3 platforms:

  1. Verified that the Sponsored Tiles are displayed on certain regions (UK, CA, IT, AU, US and DE).
  2. Verified that the Sponsored Tiles redirect to the correct page.
  3. Verified that the impression/click pings are registered in the Browser Console.
  4. Verified that the Sponsored Tiles cannot be moved to another position.
  5. Verified each option from the Sponsored Tile menu (...).
  6. Verified that the Sponsored Tiles can be dismissed.
  7. Verified that the Sponsored Tiles are displayed if a New Window is opened.
  8. Verified that if moving Non-Sponsored Tiles nothing is broken in the New Tab’s Page UI.

Besides these scenarios, we have also performed exploratory testing and we haven't found any regressions.

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

Attachment

General

Created:
Updated:
Size: