Bug 1802934 - Remove unused parameter aUsePrivateBrowsing from ParentChannelListeners constructor r=necko-reviewers,valentin

Parameter got unused in https://phabricator.services.mozilla.com/D74434

Differential Revision: https://phabricator.services.mozilla.com/D166181
This commit is contained in:
Manuel Bucher 2023-01-09 09:43:17 +00:00
parent b0759509dd
commit 1852a95106
5 changed files with 5 additions and 10 deletions

View File

@ -408,8 +408,7 @@ DocumentLoadListener::DocumentLoadListener(
: mIsDocumentLoad(aIsDocumentLoad) { : mIsDocumentLoad(aIsDocumentLoad) {
LOG(("DocumentLoadListener ctor [this=%p]", this)); LOG(("DocumentLoadListener ctor [this=%p]", this));
mParentChannelListener = mParentChannelListener =
new ParentChannelListener(this, aLoadingBrowsingContext, new ParentChannelListener(this, aLoadingBrowsingContext);
aLoadingBrowsingContext->UsePrivateBrowsing());
} }
DocumentLoadListener::~DocumentLoadListener() { DocumentLoadListener::~DocumentLoadListener() {

View File

@ -390,7 +390,7 @@ nsresult EarlyHintPreloader::OpenChannel(
success = httpChannel->SetRequestHeader("X-Moz"_ns, "early hint"_ns, false); success = httpChannel->SetRequestHeader("X-Moz"_ns, "early hint"_ns, false);
MOZ_ASSERT(NS_SUCCEEDED(success)); MOZ_ASSERT(NS_SUCCEEDED(success));
mParentListener = new ParentChannelListener(this, nullptr, false); mParentListener = new ParentChannelListener(this, nullptr);
PriorizeAsPreload(); PriorizeAsPreload();

View File

@ -536,8 +536,7 @@ bool HttpChannelParent::DoAsyncOpen(
} }
RefPtr<ParentChannelListener> parentListener = new ParentChannelListener( RefPtr<ParentChannelListener> parentListener = new ParentChannelListener(
this, mBrowserParent ? mBrowserParent->GetBrowsingContext() : nullptr, this, mBrowserParent ? mBrowserParent->GetBrowsingContext() : nullptr);
mLoadContext && mLoadContext->UsePrivateBrowsing());
httpChannel->SetRequestMethod(nsDependentCString(requestMethod.get())); httpChannel->SetRequestMethod(nsDependentCString(requestMethod.get()));

View File

@ -15,7 +15,6 @@
#include "mozilla/net/HttpChannelParent.h" #include "mozilla/net/HttpChannelParent.h"
#include "mozilla/net/RedirectChannelRegistrar.h" #include "mozilla/net/RedirectChannelRegistrar.h"
#include "mozilla/SchedulerGroup.h" #include "mozilla/SchedulerGroup.h"
#include "mozilla/Unused.h"
#include "nsIHttpHeaderVisitor.h" #include "nsIHttpHeaderVisitor.h"
#include "nsIPrompt.h" #include "nsIPrompt.h"
#include "nsISecureBrowserUI.h" #include "nsISecureBrowserUI.h"
@ -30,7 +29,6 @@
#include "mozilla/dom/LoadURIOptionsBinding.h" #include "mozilla/dom/LoadURIOptionsBinding.h"
#include "nsIWebNavigation.h" #include "nsIWebNavigation.h"
using mozilla::Unused;
using mozilla::dom::ServiceWorkerInterceptController; using mozilla::dom::ServiceWorkerInterceptController;
namespace mozilla { namespace mozilla {
@ -38,7 +36,7 @@ namespace net {
ParentChannelListener::ParentChannelListener( ParentChannelListener::ParentChannelListener(
nsIStreamListener* aListener, nsIStreamListener* aListener,
dom::CanonicalBrowsingContext* aBrowsingContext, bool aUsePrivateBrowsing) dom::CanonicalBrowsingContext* aBrowsingContext)
: mNextListener(aListener), mBrowsingContext(aBrowsingContext) { : mNextListener(aListener), mBrowsingContext(aBrowsingContext) {
LOG(("ParentChannelListener::ParentChannelListener [this=%p, next=%p]", this, LOG(("ParentChannelListener::ParentChannelListener [this=%p, next=%p]", this,
aListener)); aListener));

View File

@ -46,8 +46,7 @@ class ParentChannelListener final : public nsIInterfaceRequestor,
explicit ParentChannelListener( explicit ParentChannelListener(
nsIStreamListener* aListener, nsIStreamListener* aListener,
dom::CanonicalBrowsingContext* aBrowsingContext, dom::CanonicalBrowsingContext* aBrowsingContext);
bool aUsePrivateBrowsing);
// Called to set a new listener which replaces the old one after a redirect. // Called to set a new listener which replaces the old one after a redirect.
void SetListenerAfterRedirect(nsIStreamListener* aListener); void SetListenerAfterRedirect(nsIStreamListener* aListener);