Commit f54a0c00 authored by henry's avatar henry Committed by Pier Angelo Vendrame
Browse files

Bug 31575: Disable Firefox Home (Activity Stream) 

Treat about:blank as the default home page and new tab page.

Avoid loading AboutNewTab in BrowserGlue.sys.mjs in order
to avoid several network requests that we do not need.

Bug 41624: Disable about:pocket-* pages.

Bug 40144: Redirect about:privatebrowsing to the user's home
parent e77e19da

browser/base/content/browser.js

+0 −3
Original line number Diff line number Diff line
@@ -54,7 +54,6 @@ ChromeUtils.defineESModuleGetters(this, { 
  ReaderMode: "resource://gre/modules/ReaderMode.sys.mjs",

  SafeBrowsing: "resource://gre/modules/SafeBrowsing.sys.mjs",

  Sanitizer: "resource:///modules/Sanitizer.sys.mjs",

  SaveToPocket: "chrome://pocket/content/SaveToPocket.sys.mjs",

  ScreenshotsUtils: "resource:///modules/ScreenshotsUtils.sys.mjs",

  SearchUIUtils: "resource:///modules/SearchUIUtils.sys.mjs",

  SessionStartup: "resource:///modules/sessionstore/SessionStartup.sys.mjs",
@@ -5351,8 +5350,6 @@ var XULBrowserWindow = { 


    SafeBrowsingNotificationBox.onLocationChange(aLocationURI);



    SaveToPocket.onLocationChange(window);



    let originalURI;

    if (aRequest instanceof Ci.nsIChannel) {

      originalURI = aRequest.originalURI;

browser/components/BrowserGlue.sys.mjs

+2 −90
Original line number Diff line number Diff line
@@ -10,8 +10,6 @@ const lazy = {}; 
// Ignore unused lazy property for PluginManager.

// eslint-disable-next-line mozilla/valid-lazy

ChromeUtils.defineESModuleGetters(lazy, {

  ASRouterNewTabHook:

    "resource://activity-stream/lib/ASRouterNewTabHook.sys.mjs",

  ActorManagerParent: "resource://gre/modules/ActorManagerParent.sys.mjs",

  AddonManager: "resource://gre/modules/AddonManager.sys.mjs",

  AppMenuNotifications: "resource://gre/modules/AppMenuNotifications.sys.mjs",
@@ -60,7 +58,6 @@ ChromeUtils.defineESModuleGetters(lazy, { 
  RemoteSettings: "resource://services-settings/remote-settings.sys.mjs",

  SafeBrowsing: "resource://gre/modules/SafeBrowsing.sys.mjs",

  Sanitizer: "resource:///modules/Sanitizer.sys.mjs",

  SaveToPocket: "chrome://pocket/content/SaveToPocket.sys.mjs",

  ScreenshotsUtils: "resource:///modules/ScreenshotsUtils.sys.mjs",

  SearchSERPTelemetry: "resource:///modules/SearchSERPTelemetry.sys.mjs",

  SessionStartup: "resource:///modules/sessionstore/SessionStartup.sys.mjs",
@@ -80,9 +77,6 @@ ChromeUtils.defineESModuleGetters(lazy, { 
});



XPCOMUtils.defineLazyModuleGetters(lazy, {

  AboutNewTab: "resource:///modules/AboutNewTab.jsm",

  ASRouterDefaultConfig:

    "resource://activity-stream/lib/ASRouterDefaultConfig.jsm",

  ASRouter: "resource://activity-stream/lib/ASRouter.jsm",

  BrowserUsageTelemetry: "resource:///modules/BrowserUsageTelemetry.jsm",

  BrowserUIUtils: "resource:///modules/BrowserUIUtils.jsm",
@@ -418,31 +412,6 @@ let JSWINDOWACTORS = { 
    matches: ["about:messagepreview", "about:messagepreview?*"],

  },



  AboutNewTab: {

    parent: {

      esModuleURI: "resource:///actors/AboutNewTabParent.sys.mjs",

    },

    child: {

      esModuleURI: "resource:///actors/AboutNewTabChild.sys.mjs",

      events: {

        DOMDocElementInserted: {},

        DOMContentLoaded: {},

        load: { capture: true },

        unload: { capture: true },

        pageshow: {},

        visibilitychange: {},

      },

    },

    // The wildcard on about:newtab is for the ?endpoint query parameter

    // that is used for snippets debugging. The wildcard for about:home

    // is similar, and also allows for falling back to loading the

    // about:home document dynamically if an attempt is made to load

    // about:home?jscache from the AboutHomeStartupCache as a top-level

    // load.

    matches: ["about:home*", "about:welcome", "about:newtab*"],

    remoteTypes: ["privilegedabout"],

  },



  AboutPlugins: {

    parent: {

      esModuleURI: "resource:///actors/AboutPluginsParent.sys.mjs",
@@ -457,26 +426,6 @@ let JSWINDOWACTORS = { 


    matches: ["about:plugins"],

  },

  AboutPocket: {

    parent: {

      esModuleURI: "resource:///actors/AboutPocketParent.sys.mjs",

    },

    child: {

      esModuleURI: "resource:///actors/AboutPocketChild.sys.mjs",



      events: {

        DOMDocElementInserted: { capture: true },

      },

    },



    remoteTypes: ["privilegedabout"],

    matches: [

      "about:pocket-saved*",

      "about:pocket-signup*",

      "about:pocket-home*",

      "about:pocket-style-guide*",

    ],

  },



  AboutPrivateBrowsing: {

    parent: {
@@ -912,27 +861,6 @@ let JSWINDOWACTORS = { 
    matches: ["about:studies*"],

  },



  ASRouter: {

    parent: {

      esModuleURI: "resource:///actors/ASRouterParent.sys.mjs",

    },

    child: {

      esModuleURI: "resource:///actors/ASRouterChild.sys.mjs",

      events: {

        // This is added so the actor instantiates immediately and makes

        // methods available to the page js on load.

        DOMDocElementInserted: {},

      },

    },

    matches: [

      "about:home*",

      "about:newtab*",

      "about:welcome*",

      "about:privatebrowsing*",

    ],

    remoteTypes: ["privilegedabout"],

  },



  SwitchDocumentDirection: {

    child: {

      esModuleURI: "resource:///actors/SwitchDocumentDirectionChild.sys.mjs",
@@ -1471,8 +1399,6 @@ BrowserGlue.prototype = { 
      lazy.Normandy.init();

    }



    lazy.SaveToPocket.init();



    AboutHomeStartupCache.init();



    Services.obs.notifyObservers(null, "browser-ui-startup-complete");
@@ -1802,8 +1728,6 @@ BrowserGlue.prototype = { 


  // the first browser window has finished initializing

  _onFirstWindowLoaded: function BG__onFirstWindowLoaded(aWindow) {

    lazy.AboutNewTab.init();



    lazy.TabCrashHandler.init();



    lazy.ProcessHangMonitor.init();
@@ -2149,7 +2073,6 @@ BrowserGlue.prototype = { 
        }

      },

      () => lazy.RFPHelper.uninit(),

      () => lazy.ASRouterNewTabHook.destroy(),

      () => {

        if (AppConstants.MOZ_UPDATER) {

          lazy.UpdateListener.reset();
@@ -2946,13 +2869,6 @@ BrowserGlue.prototype = { 
        },

      },



      {

        name: "ASRouterNewTabHook.createInstance",

        task: () => {

          lazy.ASRouterNewTabHook.createInstance(lazy.ASRouterDefaultConfig());

        },

      },



      {

        name: "BackgroundUpdate",

        condition: AppConstants.MOZ_UPDATE_AGENT,
@@ -6197,12 +6113,8 @@ export var AboutHomeStartupCache = { 
      return { pageInputStream: null, scriptInputStream: null };

    }



    let state = lazy.AboutNewTab.activityStream.store.getState();

    return new Promise(resolve => {

      this._cacheDeferred = resolve;

      this.log.trace("Parent is requesting cache streams.");

      this._procManager.sendAsyncMessage(this.CACHE_REQUEST_MESSAGE, { state });

    });

    this.log.error("Activity Stream is disabled.");

    return { pageInputStream: null, scriptInputStream: null };

  },



  /**

browser/components/about/AboutRedirector.cpp

+5 −2
Original line number Diff line number Diff line
@@ -72,7 +72,7 @@ static const RedirEntry kRedirMap[] = { 
         nsIAboutModule::IS_SECURE_CHROME_UI},

    {"policies", "chrome://browser/content/policies/aboutPolicies.html",

     nsIAboutModule::ALLOW_SCRIPT | nsIAboutModule::IS_SECURE_CHROME_UI},

    {"privatebrowsing", "chrome://browser/content/aboutPrivateBrowsing.html",

    {"privatebrowsing", "about:blank",

     nsIAboutModule::URI_SAFE_FOR_UNTRUSTED_CONTENT |

         nsIAboutModule::URI_MUST_LOAD_IN_CHILD | nsIAboutModule::ALLOW_SCRIPT |

         nsIAboutModule::URI_CAN_LOAD_IN_PRIVILEGEDABOUT_PROCESS},
@@ -108,6 +108,7 @@ static const RedirEntry kRedirMap[] = { 
     nsIAboutModule::URI_MUST_LOAD_IN_CHILD |

         nsIAboutModule::URI_SAFE_FOR_UNTRUSTED_CONTENT |

         nsIAboutModule::ALLOW_SCRIPT | nsIAboutModule::HIDE_FROM_ABOUTABOUT},

#ifndef BASE_BROWSER_VERSION

    {"pocket-saved", "chrome://pocket/content/panels/saved.html",

     nsIAboutModule::URI_SAFE_FOR_UNTRUSTED_CONTENT |

         nsIAboutModule::URI_MUST_LOAD_IN_CHILD | nsIAboutModule::ALLOW_SCRIPT |
@@ -128,6 +129,7 @@ static const RedirEntry kRedirMap[] = { 
         nsIAboutModule::URI_MUST_LOAD_IN_CHILD | nsIAboutModule::ALLOW_SCRIPT |

         nsIAboutModule::URI_CAN_LOAD_IN_PRIVILEGEDABOUT_PROCESS |

         nsIAboutModule::HIDE_FROM_ABOUTABOUT},

#endif

    {"preferences", "chrome://browser/content/preferences/preferences.xhtml",

     nsIAboutModule::ALLOW_SCRIPT | nsIAboutModule::IS_SECURE_CHROME_UI},

    {"downloads",
@@ -192,7 +194,8 @@ AboutRedirector::NewChannel(nsIURI* aURI, nsILoadInfo* aLoadInfo, 
      // enabled about:newtab. Disabled about:newtab page uses fallback.

      if (path.EqualsLiteral("home") ||

          (StaticPrefs::browser_newtabpage_enabled() &&

           path.EqualsLiteral("newtab"))) {

           path.EqualsLiteral("newtab")) ||

          path.EqualsLiteral("privatebrowsing")) {

        nsCOMPtr<nsIAboutNewTabService> aboutNewTabService =

            do_GetService("@mozilla.org/browser/aboutnewtab-service;1", &rv);

        NS_ENSURE_SUCCESS(rv, rv);

browser/components/about/components.conf

+0 −4
Original line number Diff line number Diff line
@@ -14,10 +14,6 @@ pages = [ 
    'loginsimportreport',

    'messagepreview',

    'newtab',

    'pocket-home',

    'pocket-saved',

    'pocket-signup',

    'pocket-style-guide',

    'policies',

    'preferences',

    'privatebrowsing',

browser/components/moz.build

+0 −1
Original line number Diff line number Diff line
@@ -42,7 +42,6 @@ DIRS += [ 
    "originattributes",

    "pagedata",

    "places",

    "pocket",

    "preferences",

    "privatebrowsing",

    "prompts",