Quick fix for regression in service workers by reverting backports

This backport causes a crash under certain circumstances. The issue
that was being fixed was not a high priority, so reverting until a
proper fix has been found.

This reverts commits 155c60b9db6b07e16dab7b105865d49ecad2fa84,
e67dc64220e00d27be338f507c0fb4872fecdaf1 as well as
6f1275cb40ffbc83dd7b216d554d2765c239144d.

Change-Id: Id8ce09cc7115991fa176c588103541a7a52fcd39
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>

1 files changed, 0 insertions, 12 deletions

diff --git a/chromium/services/network/cors/cors_url_loader.cc b/chromium/services/network/cors/cors_url_loader.cc
index 7606ceb59fb..266520702c0 100644
--- a/chromium/services/network/cors/cors_url_loader.cc
+++ b/chromium/services/network/cors/cors_url_loader.cc
@@ -129,18 +129,6 @@ void CorsURLLoader::FollowRedirect(
     const net::HttpRequestHeaders& modified_headers,
     const net::HttpRequestHeaders& modified_cors_exempt_headers,
     const base::Optional<GURL>& new_url) {
-  // If this is a navigation from a renderer, then its a service worker
-  // passthrough of a navigation request.  Since this case uses manual
-  // redirect mode FollowRedirect() should never be called.
-  if (process_id_ != mojom::kBrowserProcessId &&
-      request_.mode == mojom::RequestMode::kNavigate) {
-    mojo::ReportBadMessage(
-        "CorsURLLoader: navigate from non-browser-process should not call "
-        "FollowRedirect");
-    HandleComplete(URLLoaderCompletionStatus(net::ERR_FAILED));
-    return;
-  }
-
   if (!network_loader_ || !deferred_redirect_url_) {
     HandleComplete(URLLoaderCompletionStatus(net::ERR_FAILED));
     return;