Pier Angelo Vendrame pushed to branch tor-browser-128.9.0esr-14.5-1 at The Tor Project / Applications / Tor Browser
Commits:
- 
a6eda23a
by Henry Wilkes at 2025-04-08T17:23:52+00:00
 - 
4ef1a8dd
by Henry Wilkes at 2025-04-08T17:23:52+00:00
 - 
bb06f3c6
by Henry Wilkes at 2025-04-08T17:23:52+00:00
 
3 changed files:
- browser/components/BrowserContentHandler.sys.mjs
 - browser/components/BrowserGlue.sys.mjs
 - browser/components/abouttor/AboutTorMessage.sys.mjs
 
Changes:
| ... | ... | @@ -876,18 +876,12 @@ nsBrowserContentHandler.prototype = { | 
| 876 | 876 |                "%OLD_BASE_BROWSER_VERSION%",
 | 
| 877 | 877 |                old_forkVersion
 | 
| 878 | 878 |              );
 | 
| 879 | -            if (overridePage && AppConstants.BASE_BROWSER_UPDATE) {
 | 
|
| 879 | +            if (AppConstants.BASE_BROWSER_UPDATE) {
 | 
|
| 880 | 880 |                // Tor Browser: Instead of opening the post-update "override page"
 | 
| 881 | 881 |                // directly, we ensure that about:tor will be opened, which should
 | 
| 882 | 882 |                // notify the user that their browser was updated.
 | 
| 883 | -              //
 | 
|
| 884 | -              // The overridePage comes from the openURL attribute within the
 | 
|
| 885 | -              // updates.xml file or, if no showURL action is present, from the
 | 
|
| 886 | -              // startup.homepage_override_url pref.
 | 
|
| 887 | -              Services.prefs.setCharPref(
 | 
|
| 888 | -                "torbrowser.post_update.url",
 | 
|
| 889 | -                overridePage
 | 
|
| 890 | -              );
 | 
|
| 883 | +              // NOTE: We ignore any overridePage value, which can come from the
 | 
|
| 884 | +              // openURL attribute within the updates.xml file.
 | 
|
| 891 | 885 |                Services.prefs.setBoolPref(
 | 
| 892 | 886 |                  "torbrowser.post_update.shouldNotify",
 | 
| 893 | 887 |                  true
 | 
| ... | ... | @@ -4830,7 +4830,9 @@ BrowserGlue.prototype = { | 
| 4830 | 4830 |      // Version 6: Tor Browser 14.5a3: Clear preference for TorSettings that is
 | 
| 4831 | 4831 |      //            no longer used (tor-browser#41921).
 | 
| 4832 | 4832 |      //            Drop unused TorConnect setting (tor-browser#43462).
 | 
| 4833 | -    const TBB_MIGRATION_VERSION = 6;
 | 
|
| 4833 | +    // Version 6: Tor Browser 14.5a6: Clear home page update url preference
 | 
|
| 4834 | +    //            (tor-browser#43567).
 | 
|
| 4835 | +    const TBB_MIGRATION_VERSION = 7;
 | 
|
| 4834 | 4836 |      const MIGRATION_PREF = "torbrowser.migration.version";
 | 
| 4835 | 4837 | |
| 4836 | 4838 |      // If we decide to force updating users to pass through any version
 | 
| ... | ... | @@ -4917,6 +4919,10 @@ BrowserGlue.prototype = { | 
| 4917 | 4919 |        Services.prefs.clearUserPref("torbrowser.bootstrap.allow_internet_test");
 | 
| 4918 | 4920 |      }
 | 
| 4919 | 4921 | |
| 4922 | +    if (currentVersion < 7) {
 | 
|
| 4923 | +      Services.prefs.clearUserPref("torbrowser.post_update.url");
 | 
|
| 4924 | +    }
 | 
|
| 4925 | +  | 
|
| 4920 | 4926 |      Services.prefs.setIntPref(MIGRATION_PREF, TBB_MIGRATION_VERSION);
 | 
| 4921 | 4927 |    },
 | 
| 4922 | 4928 | 
| ... | ... | @@ -26,13 +26,20 @@ export const AboutTorMessage = { | 
| 26 | 26 |      const shouldNotifyPref = "torbrowser.post_update.shouldNotify";
 | 
| 27 | 27 |      if (Services.prefs.getBoolPref(shouldNotifyPref, false)) {
 | 
| 28 | 28 |        Services.prefs.clearUserPref(shouldNotifyPref);
 | 
| 29 | +      // Try use the same URL as the about dialog. See tor-browser#43567.
 | 
|
| 30 | +      let updateURL = Services.urlFormatter.formatURLPref(
 | 
|
| 31 | +        "app.releaseNotesURL.aboutDialog"
 | 
|
| 32 | +      );
 | 
|
| 33 | +      if (updateURL === "about:blank") {
 | 
|
| 34 | +        updateURL = Services.urlFormatter.formatURLPref(
 | 
|
| 35 | +          "startup.homepage_override_url"
 | 
|
| 36 | +        );
 | 
|
| 37 | +      }
 | 
|
| 29 | 38 |        return {
 | 
| 30 | 39 |          updateVersion: Services.prefs.getCharPref(
 | 
| 31 | 40 |            "browser.startup.homepage_override.torbrowser.version"
 | 
| 32 | 41 |          ),
 | 
| 33 | -        updateURL:
 | 
|
| 34 | -          Services.prefs.getCharPref("torbrowser.post_update.url", "") ||
 | 
|
| 35 | -          Services.urlFormatter.formatURLPref("startup.homepage_override_url"),
 | 
|
| 42 | +        updateURL,
 | 
|
| 36 | 43 |        };
 | 
| 37 | 44 |      }
 | 
| 38 | 45 |      const number = this._count;
 |