tor-browser

commit f90a6d1710e68787ea18296790cd4343804e79bb
parent 32d80d7cb1efb62547f39f47decbb85dcadb663f
Author: scottdowne <sdowne@mozilla.com>
Date:   Sun,  2 Nov 2025 14:07:31 +0000

Bug 1997658 - Newtab remove discovery stream spocs from shortcuts feed r=maxx

Differential Revision: https://phabricator.services.mozilla.com/D270911

Diffstat:
M
browser/components/newtab/test/browser/abouthomecache/browser_no_startup_actions.js
 | 
10
++--------
M
browser/extensions/newtab/lib/TopSitesFeed.sys.mjs
 | 
5
+----

2 files changed, 3 insertions(+), 12 deletions(-)
diff --git a/browser/components/newtab/test/browser/abouthomecache/browser_no_startup_actions.js b/browser/components/newtab/test/browser/abouthomecache/browser_no_startup_actions.js
@@ -70,23 +70,17 @@ add_task(async function test_no_startup_actions() {
 
     Assert.equal(
       nonStartupActions.length,
-      2,
+      1,
       "Should be no non-ads non-startup actions."
     );
 
-    // We expect 2 startup actions for ads.
+    // We expect 1 startup action for ads.
     Assert.equal(
       nonStartupActions[0].type,
       "DISCOVERY_STREAM_SPOCS_UPDATE",
       "Should be a single DISCOVERY_STREAM_SPOCS_UPDATE action"
     );
 
-    Assert.equal(
-      nonStartupActions[1].type,
-      "TOP_SITES_UPDATED",
-      "Should be a single TOP_SITES_UPDATED action"
-    );
-
     if (nonStartupActions.length) {
       for (let action of nonStartupActions) {
         info(`Non-startup action: ${action.type}`);
diff --git a/browser/extensions/newtab/lib/TopSitesFeed.sys.mjs b/browser/extensions/newtab/lib/TopSitesFeed.sys.mjs
@@ -2095,6 +2095,7 @@ export class TopSitesFeed {
         this.updateCustomSearchShortcuts(true /* isStartup */);
         break;
       case at.DISCOVERY_STREAM_DEV_SYSTEM_TICK:
+      case at.DISCOVERY_STREAM_DEV_EXPIRE_CACHE:
       case at.SYSTEM_TICK:
         this.refresh({ broadcast: false });
         this._contile.periodicUpdate();
@@ -2178,10 +2179,6 @@ export class TopSitesFeed {
       case at.ADS_UPDATE_TILES:
         this._contile.refresh();
         break;
-      case at.DISCOVERY_STREAM_SPOCS_UPDATE:
-        // Refresh to update sponsored topsites.
-        this.refresh({ broadcast: true, isStartup: action.meta.isStartup });
-        break;
       case at.UNINIT:
         this.uninit();
         break;