654287 - nsDocShell.cpp:5551:68: warning: 'virtual nsresult nsIScriptSecurityManager::CheckLoadURI(nsIURI*, nsIURI*, PRUint32)' is deprecated

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Daniel Holbert [:dholbert]]

Filing bug on this build warning:
{
nsDocShell.cpp: In member function 'virtual nsresult nsDocShell::SetupRefreshURIFromHeader(nsIURI*, const nsACString_internal&)':

nsDocShell.cpp:5551:68: warning: 'virtual nsresult nsIScriptSecurityManager::CheckLoadURI(nsIURI*, nsIURI*, PRUint32)' is deprecated (declared at ../../dist/include/nsIScriptSecurityManager.h:101)
}

Specifically, the CheckLoadURI documentation says:
>  @deprecated Use checkLoadURIWithPrincipal instead of this function.

Code references:
http://mxr.mozilla.org/mozilla-central/source/docshell/base/nsDocShell.cpp#5548
http://mxr.mozilla.org/mozilla-central/source/caps/idl/nsIScriptSecurityManager.idl#105

Comment 1

[Daniel Holbert [:dholbert]]

(oops; initially added dependency on bug 120373 since that's the last bug to touch these lines of code (in bonsai history), but it looks like that bug merely tweaked code that had been added much much earlier, before CheckLoadURI was deprecated)

Comment 2

[:Cykesiopka]

Fixed by Bug 327244.