Bug 1701668, remove shouldLoadURI and shouldLoadURIInThisProcess methods now that all implementers return true, r=kmag

Differential Revision: https://phabricator.services.mozilla.com/D110101

docshell/base/nsDocShell.cpp

@@ -9525,22 +9525,6 @@ nsresult nsDocShell::InternalLoad(nsDocShellLoadState* aLoadState,
     mTiming->NotifyUnloadAccepted(mCurrentURI);
   }
 
-  // Check if the webbrowser chrome wants the load to proceed; this can be
-  // used to cancel attempts to load URIs in the wrong process. testing
-  // GetPendingRedirectedChannel() helps to avoid revisiting an earlier
-  // redirect decision.
-  nsCOMPtr<nsIWebBrowserChrome3> browserChrome3 = do_GetInterface(mTreeOwner);
-  if (browserChrome3 && !aLoadState->GetPendingRedirectedChannel()) {
-    bool shouldLoad;
-    rv = browserChrome3->ShouldLoadURI(
-        this, aLoadState->URI(), aLoadState->GetReferrerInfo(),
-        !!aLoadState->PostDataStream(), aLoadState->TriggeringPrincipal(),
-        aLoadState->Csp(), &shouldLoad);
-    if (NS_SUCCEEDED(rv) && !shouldLoad) {
-      return NS_OK;
-    }
-  }
-
   // In e10s, in the parent process, we refuse to load anything other than
   // "safe" resources that we ship or trust enough to give "special" URLs.
   // Similar check will be performed by the ParentProcessDocumentChannel if in