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

Bug 1490257 - Add asserts into loadURI where we imply SystemPrincipal. r=ckerschb

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D10221
This commit is contained in:
Jonathan Kingston
2018-10-31 18:00:40 +00:00
parent 470a812368
commit b25eb46164
38 changed files with 128 additions and 55 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
toolkit/components/browser/nsWebBrowser.cpp
View File

@@ -512,6 +512,9 @@ nsWebBrowser::LoadURIWithOptions(const nsAString& aURI, uint32_t aLoadFlags,
nsIURI* aBaseURI,
nsIPrincipal* aTriggeringPrincipal)
{
#ifndef ANDROID
MOZ_ASSERT(aTriggeringPrincipal, "nsWebBrowser::LoadURIWithOptions - Need a valid triggeringPrincipal");
#endif
NS_ENSURE_STATE(mDocShell);
return mDocShellAsNav->LoadURIWithOptions(
@@ -533,6 +536,9 @@ nsWebBrowser::LoadURI(const nsAString& aURI, uint32_t aLoadFlags,
nsIInputStream* aExtraHeaderStream,
nsIPrincipal* aTriggeringPrincipal)
{
#ifndef ANDROID
MOZ_ASSERT(aTriggeringPrincipal, "nsWebBrowser::LoadURI - Need a valid triggeringPrincipal");
#endif
NS_ENSURE_STATE(mDocShell);
return mDocShellAsNav->LoadURI(aURI, aLoadFlags, aReferringURI,