corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: allow BUILD_ID changes to show release notes

Browse Source 
(cherry picked from commit 87b3885bb425a91e27a4ec9ef773afe0e428be95)
This commit is contained in:
Alex Kontos
2023-10-05 17:12:10 +01:00
parent 6d4862fcc8
commit 7383a3518b
1 changed files with 30 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
browser/components/BrowserContentHandler.sys.mjs
View File

@@ -745,6 +745,7 @@ nsBrowserContentHandler.prototype = {
// to retrieve any other startup pages that needs to be displayed. // to retrieve any other startup pages that needs to be displayed.
// See Bug 1642039 for more information. // See Bug 1642039 for more information.
getFirstWindowArgs() { getFirstWindowArgs() {
var prefb = Services.prefs; var prefb = Services.prefs;
if (!gFirstWindow) { if (!gFirstWindow) {
@@ -918,6 +919,7 @@ nsBrowserContentHandler.prototype = {
lazy.LaterRun.enable(lazy.LaterRun.ENABLE_REASON_UPDATE_APPLIED); lazy.LaterRun.enable(lazy.LaterRun.ENABLE_REASON_UPDATE_APPLIED);
} }
// Send the update ping to signal that the update was successful. // Send the update ping to signal that the update was successful.
// Only do this if the update is installed right now. // Only do this if the update is installed right now.
// The following code is ran asynchronously, but we won't await on it // The following code is ran asynchronously, but we won't await on it
@@ -947,11 +949,17 @@ nsBrowserContentHandler.prototype = {
break; break;
} }
case OVERRIDE_NEW_BUILD_ID: { case OVERRIDE_NEW_BUILD_ID: {
// We must spin the events loop because `getFirstWindowArgs` cannot be // We must spin the events loop because `getFirstWindowArgs` cannot be
// easily made asynchronous, having too many synchronous callers. Additionally // easily made asynchronous, having too many synchronous callers. Additionally
// we must know the value of `updateInstalledAtStartup` immediately, // we must know the value of `updateInstalledAtStartup` immediately,
// in order to properly enable `lazy.LaterRun`, that will be invoked shortly after this. // in order to properly enable `lazy.LaterRun`, that will be invoked shortly after this.
let updateInstalledAtStartup = spinForUpdateInstalledAtStartup(); let updateInstalledAtStartup = false;
try {
updateInstalledAtStartup = spinForUpdateInstalledAtStartup();
} catch (e) {
// ignore
}
if (updateInstalledAtStartup) { if (updateInstalledAtStartup) {
let handleUpdateSuccessTask = lazy.UpdatePing.handleUpdateSuccess( let handleUpdateSuccessTask = lazy.UpdatePing.handleUpdateSuccess(
@@ -968,6 +976,27 @@ nsBrowserContentHandler.prototype = {
lazy.LaterRun.enable(lazy.LaterRun.ENABLE_REASON_UPDATE_APPLIED); lazy.LaterRun.enable(lazy.LaterRun.ENABLE_REASON_UPDATE_APPLIED);
} }
// Open the release notes / What's New Page for build-id-only updates,
// regardless of whether the update manager reports it was installed at startup.
// Prefer update-provided URL when allowed; fall back to the pref.
try {
const defaultOverridePage =
Services.urlFormatter.formatURLPref("startup.homepage_override_url");
const update = spinForLastUpdateInstalled();
// getPostUpdateOverridePage enforces actions/policy and can return "" if disabled.
overridePage = getPostUpdateOverridePage(update, defaultOverridePage);
// Fallback to pref-defined page if update actions/policy suppressed WNP.
if (!overridePage) {
overridePage = defaultOverridePage;
}
} catch (e) {
// If update info isn't available, just use the pref directly.
overridePage =
Services.urlFormatter.formatURLPref("startup.homepage_override_url");
}
overridePage = overridePage.replace("%OLD_VERSION%", old_mstone);
break; break;
} }
} }