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;
|