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

Bug 780562 - Remove support for the chrome "platform" flag. r=bsmedberg

Browse Source 
Now that the sole use of the flag was removed, remove support for it.
This commit is contained in:
Mike Hommey
2017-04-14 20:48:53 +09:00
parent 9f74f4441c
commit a60a839fe4
3 changed files with 1 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
xpcom/components/ManifestParser.cpp
View File

@@ -56,7 +56,7 @@ struct ManifestDirective
bool allowbootstrap;
// The platform/contentaccessible flags only apply to content directives.
// The contentaccessible flags only apply to content directives.
bool contentflags;
// Function to handle this directive. This isn't a union because C++ still
@@ -449,7 +449,6 @@ ParseManifest(NSLocationType aType, FileLocation& aFile, char* aBuf,
nsChromeRegistry::ManifestProcessingContext chromecx(aType, aFile);
nsresult rv;
NS_NAMED_LITERAL_STRING(kPlatform, "platform");
NS_NAMED_LITERAL_STRING(kContentAccessible, "contentaccessible");
NS_NAMED_LITERAL_STRING(kRemoteEnabled, "remoteenabled");
NS_NAMED_LITERAL_STRING(kRemoteRequired, "remoterequired");
@@ -685,11 +684,6 @@ ParseManifest(NSLocationType aType, FileLocation& aFile, char* aBuf,
if (directive->contentflags) {
bool flag;
if (CheckFlag(kPlatform, wtoken, flag)) {
if (flag)
flags |= nsChromeRegistry::PLATFORM_PACKAGE;
continue;
}
if (CheckFlag(kContentAccessible, wtoken, flag)) {
if (flag)
flags |= nsChromeRegistry::CONTENT_ACCESSIBLE;