Revert of The Appcache subresource URL factory needs to inform the URLLoaderClient if there is a failure.

content/browser/appcache/appcache_subresource_url_factory.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/appcache/appcache_subresource_url_factory.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "content/browser/appcache/appcache_host.h"
 #include "content/browser/appcache/appcache_request_handler.h"
@@ skipping 45 matching lines @@
     int32_t request_id,
     uint32_t options,
     const ResourceRequest& request,
     mojom::URLLoaderClientPtr client,
     const net::MutableNetworkTrafficAnnotationTag& traffic_annotation) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   DLOG(WARNING) << "Received request for loading : " << request.url.spec();
 
   // If the host is invalid, it means that the renderer has probably died.
   // (Frame has navigated elsewhere?)
-  if (!appcache_host_.get()) {
-    NotifyError(std::move(client), net::ERR_FAILED);
+  if (!appcache_host_.get())
     return;
-  }
 
   std::unique_ptr<AppCacheRequestHandler> handler =
       appcache_host_->CreateRequestHandler(
           AppCacheURLLoaderRequest::Create(request), request.resource_type,
           request.should_reset_appcache);
   if (!handler) {
-    NotifyError(std::move(client), net::ERR_FAILED);
+    ResourceRequestCompletionStatus request_result;
+    request_result.error_code = net::ERR_FAILED;
+    client->OnComplete(request_result);
     return;
   }
 
   handler->set_network_url_loader_factory_getter(
       default_url_loader_factory_getter_.get());
 
   std::unique_ptr<SubresourceLoadInfo> load_info(new SubresourceLoadInfo());
   load_info->url_loader_request = std::move(url_loader_request);
   load_info->routing_id = routing_id;
   load_info->request_id = request_id;
@@ skipping 15 matching lines @@
                                              int32_t request_id,
                                              const ResourceRequest& request,
                                              SyncLoadCallback callback) {
   NOTREACHED();
 }
 
 void AppCacheSubresourceURLFactory::OnConnectionError() {
   base::ThreadTaskRunnerHandle::Get()->DeleteSoon(FROM_HERE, this);
 }
 
-void AppCacheSubresourceURLFactory::NotifyError(
-    mojom::URLLoaderClientPtr client,
-    int error_code) {
-  ResourceRequestCompletionStatus request_result;
-  request_result.error_code = error_code;
-  client->OnComplete(request_result);
-}
-
 }  // namespace content