Bug 1145470 - remove hiddenWindow specialcasing from CheckLoadURI code, r?bholley
All the consumers relying on this have disappeared, so we can (finally!) get
rid of this ugly bit of special-casing.
MozReview-Commit-ID: HSeeG21O0p4
--- a/caps/nsScriptSecurityManager.cpp
+++ b/caps/nsScriptSecurityManager.cpp
@@ -893,24 +893,16 @@ nsScriptSecurityManager::CheckLoadURIFla
bool accessAllowed = false;
reg->AllowContentToAccess(aTargetBaseURI, &accessAllowed);
if (accessAllowed) {
return NS_OK;
}
}
}
- // Special-case the hidden window: it's allowed to load
- // URI_IS_UI_RESOURCE no matter what. Bug 1145470 tracks removing this.
- nsAutoCString sourceSpec;
- if (NS_SUCCEEDED(aSourceBaseURI->GetSpec(sourceSpec)) &&
- sourceSpec.EqualsLiteral("resource://gre-resources/hiddenWindow.html")) {
- return NS_OK;
- }
-
if (reportErrors) {
ReportError(nullptr, errorTag, aSourceURI, aTargetURI);
}
return NS_ERROR_DOM_BAD_URI;
}
// Check for target URI pointing to a file
rv = NS_URIChainHasFlags(aTargetURI,