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

Bug 863246 - Content can only load resource:// URIs declared content-accessible in manifests r=billm,bz

Browse Source 
bz:
    caps/nsScriptSecurityManager.cpp

billm:
    browser/extensions/activity-stream/jar.mn
    browser/extensions/onboarding/jar.mn
    chrome/RegistryMessageUtils.h
    chrome/nsChromeRegistry.h
    chrome/nsChromeRegistryChrome.cpp
    chrome/nsChromeRegistryContent.cpp
    netwerk/protocol/res/SubstitutingProtocolHandler.cpp
    netwerk/protocol/res/SubstitutingProtocolHandler.h
    netwerk/protocol/res/nsIResProtocolHandler.idl
    netwerk/protocol/res/nsISubstitutingProtocolHandler.idl
    netwerk/protocol/res/nsResProtocolHandler.cpp
    netwerk/protocol/res/nsResProtocolHandler.h
    xpcom/components/ManifestParser.cpp

MozReview-Commit-ID: 1RXeNn7jdBf
This commit is contained in:
Chung-Sheng Fu
2017-06-08 17:44:09 +08:00
parent c423c910cd
commit 2b63f690ba
14 changed files with 183 additions and 37 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
xpcom/components/ManifestParser.cpp
View File

@@ -56,7 +56,7 @@ struct ManifestDirective
bool allowbootstrap;
// The contentaccessible flags only apply to content directives.
// The contentaccessible flags only apply to content/resource directives.
bool contentflags;
// Function to handle this directive. This isn't a union because C++ still
@@ -123,7 +123,7 @@ static const ManifestDirective kParsingTable[] = {
nullptr, &nsChromeRegistry::ManifestOverride, nullptr
},
{
"resource", 2, false, true, true, true, false,
"resource", 2, false, true, true, true, true,
nullptr, &nsChromeRegistry::ManifestResource, nullptr
}
};