wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 844243 - Installing a packaged app in which the mini-manifest app name is different than the webapp manifest app name is allowed, but should not be; r=fabrice

Browse Source
This commit is contained in:
Fernando Jiménez 2013-02-27 12:39:15 +01:00
parent a25a4df5d7
commit 7e87ab61d5
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
dom/apps/src/Webapps.jsm
View File

@ -2315,9 +2315,16 @@ this.DOMApplicationRegistry = {
throw "INVALID_MANIFEST";
}
// Call ensureSameAppName before compareManifests, as `manifest`,
// has been normalized to avoid app rename.
AppsUtils.ensureSameAppName(aManifest._manifest, manifest, app);
// For app updates we don't forbid apps to rename themselves but
// we still retain the old name of the app. In the future we
// will use UI to allow updates to rename an app after we check
// with the user that the rename is ok.
if (aIsUpdate) {
// Call ensureSameAppName before compareManifests as `manifest`
// has been normalized to avoid app rename.
AppsUtils.ensureSameAppName(aManifest._manifest, manifest,
app);
}
if (!AppsUtils.compareManifests(manifest,
aManifest._manifest)) {