Bug 1621944 - Make the top sites provider not assume the activity stream top sites pref exists. r=harry a=jcristau
authorDrew Willcoxon <adw@mozilla.com>
Fri, 20 Mar 2020 20:00:41 +0000
changeset 580627 b341f56382a48eeb01fcdb97913914ffc96e47ad
parent 580626 03f0ab1801fa4d81849cb1069b9cbdc8710cfc84
child 580628 e5b1e0d7250f84f46a48983a87ea335e030f3fd3
push id12891
push useropoprus@mozilla.com
push dateSun, 22 Mar 2020 13:22:12 +0000
treeherdermozilla-beta@e5b1e0d7250f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersharry, jcristau
bugs1621944
milestone75.0
Bug 1621944 - Make the top sites provider not assume the activity stream top sites pref exists. r=harry a=jcristau Differential Revision: https://phabricator.services.mozilla.com/D67581
browser/components/urlbar/UrlbarProviderTopSites.jsm
--- a/browser/components/urlbar/UrlbarProviderTopSites.jsm
+++ b/browser/components/urlbar/UrlbarProviderTopSites.jsm
@@ -75,17 +75,18 @@ class ProviderTopSites extends UrlbarPro
     // make this provider inactive.  For empty search strings, we instead show
     // the most frecent URLs in the user's history from the UnifiedComplete
     // provider.
     return (
       UrlbarPrefs.get("update1") &&
       UrlbarPrefs.get("openViewOnFocus") &&
       !queryContext.searchString &&
       Services.prefs.getBoolPref(
-        "browser.newtabpage.activity-stream.feeds.topsites"
+        "browser.newtabpage.activity-stream.feeds.topsites",
+        false
       )
     );
   }
 
   /**
    * Gets the provider's priority.
    * @param {UrlbarQueryContext} queryContext The query context object
    * @returns {number} The provider's priority for the given query.