зеркало из https://github.com/mozilla/gecko-dev.git
fixes bug 273078 "nsAppRunner.cpp should write compatibility.ini" r=bengoodger+bsmedberg
This commit is contained in:
darin%meer.net
Родитель
242cf81302
Коммит
ace6577c93
|
|
@ -69,7 +69,7 @@ const FILE_EXTENSIONS = "Extensions.rdf";
|
|||
const FILE_UNINSTALL_LOG = "Uninstall";
|
||||
const FILE_DEFAULTS = "defaults.ini";
|
||||
const FILE_COMPONENT_MANIFEST = "components.ini";
|
||||
const FILE_COMPAT_MANIFEST = "compatibility.ini";
|
||||
const FILE_AUTOREG = ".autoreg";
|
||||
const FILE_INSTALL_MANIFEST = "install.rdf";
|
||||
const FILE_CHROME_MANIFEST = "contents.rdf";
|
||||
const FILE_WASINSAFEMODE = "Safe Mode";
|
||||
|
|
@ -1947,7 +1947,7 @@ nsExtensionManager.prototype = {
|
|||
// If no additional restart is required, it implies that there are
|
||||
// no new components that need registering so we can inform the app
|
||||
// not to do any extra startup checking next time round.
|
||||
this._writeCompatibilityManifest(needsRestart);
|
||||
this._setComponentsListChanged(needsRestart);
|
||||
}
|
||||
catch (e) {
|
||||
dump("*** nsExtensionManager::_finishOperations - failure, catching exception so finalize window can close " + e +"\n");
|
||||
|
|
@ -2170,33 +2170,23 @@ nsExtensionManager.prototype = {
|
|||
this._writeProfileFile(manifest, this._getComponentsDir, aIsProfile);
|
||||
|
||||
// Now refresh the compatibility manifest.
|
||||
this._writeCompatibilityManifest(true);
|
||||
this._setComponentsListChanged(true);
|
||||
},
|
||||
|
||||
_writeCompatibilityManifest: function nsExtensionManager__writeCompatibilityManifest (aComponentListUpdated)
|
||||
_setComponentsListChanged: function nsExtensionManager__setComponentsListChanged (aComponentListUpdated)
|
||||
{
|
||||
var fos = Components.classes["@mozilla.org/network/file-output-stream;1"]
|
||||
.createInstance(Components.interfaces.nsIFileOutputStream);
|
||||
const MODE_WRONLY = 0x02;
|
||||
const MODE_CREATE = 0x08;
|
||||
const MODE_TRUNCATE = 0x20;
|
||||
// This code assumes that we are the only one responsible for creating this
|
||||
// file. That's OK for now, but in the future we may need to replace this
|
||||
// with code that manages a reference count or something jazzy like that.
|
||||
|
||||
// The compat file only lives in the Profile dir because we make the
|
||||
// assumption that you can never have extensions prior to profile
|
||||
// startup.
|
||||
var compat = getFile(KEY_PROFILEDIR, [FILE_COMPAT_MANIFEST]);
|
||||
fos.init(compat, MODE_WRONLY | MODE_CREATE | MODE_TRUNCATE, 0644, 0);
|
||||
|
||||
var currAppBuildID = gApp.appBuildID;
|
||||
|
||||
var val = aComponentListUpdated ? 1 : 0;
|
||||
var lines = ["[Compatibility]\r\n",
|
||||
"Build ID=" + currAppBuildID + "\r\n",
|
||||
"Components List Changed=" + val + "\r\n"];
|
||||
for (var i = 0; i < lines.length; ++i)
|
||||
fos.write(lines[i], lines[i].length);
|
||||
|
||||
fos.close();
|
||||
var autoregFile = getFile(KEY_PROFILEDIR, [FILE_AUTOREG]);
|
||||
try {
|
||||
if (aComponentListUpdated)
|
||||
autoregFile.create(Components.interfaces.nsIFile.NORMAL_FILE_TYPE, 0600);
|
||||
else
|
||||
autoregFile.remove(false);
|
||||
}
|
||||
catch (e) {}
|
||||
},
|
||||
|
||||
_writeDefaults: function nsExtensionManager__writeDefaults (aIsProfile)
|
||||
|
|
|
|||
|
|
@ -40,6 +40,7 @@
|
|||
|
||||
|
||||
#include "nsAppRunner.h"
|
||||
#include "nsBuildID.h"
|
||||
|
||||
#ifdef XP_MACOSX
|
||||
#include "MacLaunchHelper.h"
|
||||
|
|
@ -1723,47 +1724,90 @@ SelectProfile(nsIProfileLock* *aResult, nsINativeAppSupport* aNative,
|
|||
return ShowProfileManager(profileSvc, aNative);
|
||||
}
|
||||
|
||||
#define FILE_COMPATIBILITY_INFO NS_LITERAL_STRING("compatibility.ini")
|
||||
#define FILE_COMPATIBILITY_INFO NS_LITERAL_CSTRING("compatibility.ini")
|
||||
|
||||
static void GetVersion(nsIFile* aProfileDir, char* aVersion, int aVersionLength)
|
||||
{
|
||||
nsCOMPtr<nsIFile> compatibilityFile;
|
||||
aProfileDir->Clone(getter_AddRefs(compatibilityFile));
|
||||
compatibilityFile->Append(FILE_COMPATIBILITY_INFO);
|
||||
aVersion[0] = '\0';
|
||||
|
||||
nsCOMPtr<nsIFile> file;
|
||||
aProfileDir->Clone(getter_AddRefs(file));
|
||||
if (!file)
|
||||
return;
|
||||
file->AppendNative(FILE_COMPATIBILITY_INFO);
|
||||
|
||||
nsINIParser parser;
|
||||
nsCOMPtr<nsILocalFile> lf(do_QueryInterface(compatibilityFile));
|
||||
parser.Init(lf);
|
||||
nsCOMPtr<nsILocalFile> localFile(do_QueryInterface(file));
|
||||
nsresult rv = parser.Init(localFile);
|
||||
if (NS_FAILED(rv))
|
||||
return;
|
||||
|
||||
parser.GetString("Compatibility", "Build ID", aVersion, aVersionLength);
|
||||
parser.GetString("Compatibility", "LastVersion", aVersion, aVersionLength);
|
||||
}
|
||||
|
||||
static void BuildVersion(nsCString &aBuf)
|
||||
{
|
||||
aBuf.Assign(gAppData->appVersion);
|
||||
aBuf.Append('_');
|
||||
aBuf.Append(gAppData->appBuildID);
|
||||
aBuf.Append('/');
|
||||
aBuf.AppendLiteral(GRE_BUILD_ID);
|
||||
}
|
||||
|
||||
static void WriteVersion(nsIFile* aProfileDir, const nsCSubstring &version)
|
||||
{
|
||||
nsCOMPtr<nsIFile> file;
|
||||
aProfileDir->Clone(getter_AddRefs(file));
|
||||
if (!file)
|
||||
return;
|
||||
file->AppendNative(FILE_COMPATIBILITY_INFO);
|
||||
|
||||
nsCOMPtr<nsILocalFile> lf = do_QueryInterface(file);
|
||||
|
||||
PRFileDesc *fd = nsnull;
|
||||
lf->OpenNSPRFileDesc(PR_WRONLY | PR_CREATE_FILE | PR_TRUNCATE, 0600, &fd);
|
||||
if (!fd) {
|
||||
NS_ERROR("could not create output stream");
|
||||
return;
|
||||
}
|
||||
|
||||
nsCAutoString buf;
|
||||
buf.AssignLiteral("[Compatibility]\r\nLastVersion=");
|
||||
buf.Append(version);
|
||||
buf.AppendLiteral(NS_LINEBREAK);
|
||||
|
||||
PR_Write(fd, buf.get(), buf.Length());
|
||||
PR_Close(fd);
|
||||
}
|
||||
|
||||
static PRBool ComponentsListChanged(nsIFile* aProfileDir)
|
||||
{
|
||||
nsCOMPtr<nsIFile> compatibilityFile;
|
||||
aProfileDir->Clone(getter_AddRefs(compatibilityFile));
|
||||
compatibilityFile->Append(FILE_COMPATIBILITY_INFO);
|
||||
nsCOMPtr<nsIFile> file;
|
||||
aProfileDir->Clone(getter_AddRefs(file));
|
||||
if (!file)
|
||||
return PR_TRUE;
|
||||
file->AppendNative(NS_LITERAL_CSTRING(".autoreg"));
|
||||
|
||||
nsINIParser parser;
|
||||
nsCOMPtr<nsILocalFile> lf(do_QueryInterface(compatibilityFile));
|
||||
parser.Init(lf);
|
||||
|
||||
char parserBuf[MAXPATHLEN];
|
||||
nsresult rv = parser.GetString("Compatibility", "Components List Changed", parserBuf, MAXPATHLEN);
|
||||
return NS_SUCCEEDED(rv) && !strcmp(parserBuf, "1");
|
||||
PRBool exists = PR_FALSE;
|
||||
file->Exists(&exists);
|
||||
return exists;
|
||||
}
|
||||
|
||||
static void RemoveComponentRegistries(nsIFile* aProfileDir)
|
||||
{
|
||||
nsCOMPtr<nsIFile> compregFile;
|
||||
aProfileDir->Clone(getter_AddRefs(compregFile));
|
||||
compregFile->Append(NS_LITERAL_STRING("compreg.dat"));
|
||||
compregFile->Remove(PR_FALSE);
|
||||
nsCOMPtr<nsIFile> file;
|
||||
aProfileDir->Clone(getter_AddRefs(file));
|
||||
if (!file)
|
||||
return;
|
||||
|
||||
nsCOMPtr<nsIFile> xptiFile;
|
||||
aProfileDir->Clone(getter_AddRefs(xptiFile));
|
||||
xptiFile->Append(NS_LITERAL_STRING("xpti.dat"));
|
||||
xptiFile->Remove(PR_FALSE);
|
||||
file->AppendNative(NS_LITERAL_CSTRING("compreg.dat"));
|
||||
file->Remove(PR_FALSE);
|
||||
|
||||
file->SetNativeLeafName(NS_LITERAL_CSTRING("xpti.dat"));
|
||||
file->Remove(PR_FALSE);
|
||||
|
||||
file->SetNativeLeafName(NS_LITERAL_CSTRING(".autoreg"));
|
||||
file->Remove(PR_FALSE);
|
||||
}
|
||||
|
||||
const nsXREAppData* gAppData = nsnull;
|
||||
|
|
@ -2007,26 +2051,24 @@ int xre_main(int argc, char* argv[], const nsXREAppData* aAppData)
|
|||
PRBool upgraded = PR_FALSE;
|
||||
PRBool componentsListChanged = PR_FALSE;
|
||||
|
||||
if (gAppData->flags & NS_XRE_ENABLE_EXTENSION_MANAGER) {
|
||||
// Check for version compatibility with the last version of the app this
|
||||
// profile was started with.
|
||||
char version[MAXPATHLEN];
|
||||
GetVersion(lf, version, MAXPATHLEN);
|
||||
// profile was started with. The format of the version stamp is defined
|
||||
// by the BuildVersion function.
|
||||
char lastVersion[MAXPATHLEN];
|
||||
GetVersion(lf, lastVersion, MAXPATHLEN);
|
||||
|
||||
// Extensions are deemed compatible for all builds in the "x.x.x+"
|
||||
// period in between milestones for developer convenience (even though
|
||||
// ongoing code changes might actually make that a poor assumption.
|
||||
// The size and expertise of the nightly build testing community is
|
||||
// expected to be sufficient to deal with this issue.
|
||||
//
|
||||
// Every time a profile is loaded by a build with a different build id,
|
||||
// it updates the compatibility.ini file saying what build last wrote
|
||||
// the compreg.dat. On subsequent launches if the build id matches,
|
||||
// Build the version stamp for the running application.
|
||||
nsCAutoString version;
|
||||
BuildVersion(version);
|
||||
|
||||
// Every time a profile is loaded by a build with a different version,
|
||||
// it updates the compatibility.ini file saying what version last wrote
|
||||
// the compreg.dat. On subsequent launches if the version matches,
|
||||
// there is no need for re-registration. If the user loads the same
|
||||
// profile in different builds the component registry must be
|
||||
// re-generated to prevent mysterious component loading failures.
|
||||
//
|
||||
if (!strcmp(version, aAppData->appBuildID)) {
|
||||
if (version.Equals(lastVersion)) {
|
||||
componentsListChanged = ComponentsListChanged(lf);
|
||||
if (componentsListChanged) {
|
||||
// Remove compreg.dat and xpti.dat, forcing component re-registration,
|
||||
|
|
@ -2048,9 +2090,8 @@ int xre_main(int argc, char* argv[], const nsXREAppData* aAppData)
|
|||
// with a list of XPCOM components for compatible extensions
|
||||
upgraded = PR_TRUE;
|
||||
|
||||
// The Extension Manager will write the Compatibility manifest with
|
||||
// the current app version.
|
||||
}
|
||||
// Write out version
|
||||
WriteVersion(lf, version);
|
||||
}
|
||||
|
||||
PRBool needsRestart = PR_FALSE;
|
||||
|
|
|
|||
Загрузка…
Ссылка в новой задаче