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

RESOLVED FIXED

Status

()

RESOLVED FIXED
8 years ago
6 years ago

People

(Reporter: dholbert, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [build_warning])

(Reporter)

Description

8 years ago
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
(Reporter)

Updated

8 years ago
Depends on: 120373
(Reporter)

Comment 1

8 years ago
(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)
No longer depends on: 120373

Updated

7 years ago
Blocks: 187528

Comment 2

6 years ago
Fixed by Bug 327244.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.