Network service: avoid requesting a new URLLoaderFactory for each NavigationURLLoaderNetworkService.

content/browser/loader/navigation_url_loader_network_service.h

 // 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.
 
 #ifndef CONTENT_BROWSER_LOADER_NAVIGATION_URL_LOADER_NETWORK_SERVICE_H_
 #define CONTENT_BROWSER_LOADER_NAVIGATION_URL_LOADER_NETWORK_SERVICE_H_
 
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "content/browser/loader/navigation_url_loader.h"
@@ skipping 22 matching lines @@
   NavigationURLLoaderNetworkService(
       ResourceContext* resource_context,
       StoragePartition* storage_partition,
       std::unique_ptr<NavigationRequestInfo> request_info,
       std::unique_ptr<NavigationUIData> navigation_ui_data,
       ServiceWorkerNavigationHandle* service_worker_handle,
       AppCacheNavigationHandle* appcache_handle,
       NavigationURLLoaderDelegate* delegate);
   ~NavigationURLLoaderNetworkService() override;
 
-  // Overrides the URLLoaderFactory for the next request.
+  // Overrides the URLLoaderFactory for subsequent requests. Passing a null
+  // pointer will restore the default behavior.
   static CONTENT_EXPORT void OverrideURLLoaderFactoryForTesting(
       mojom::URLLoaderFactoryPtr url_loader_factory);
 
   // NavigationURLLoader implementation:
   void FollowRedirect() override;
   void ProceedWithResponse() override;
 
   // mojom::URLLoaderClient implementation:
   void OnReceiveResponse(const ResourceResponseHead& head,
                          const base::Optional<net::SSLInfo>& ssl_info,
@@ skipping 11 matching lines @@
   void OnComplete(
       const ResourceRequestCompletionStatus& completion_status) override;
 
   // Initiates the request.
   void StartURLRequest(std::unique_ptr<ResourceRequest> request);
 
  private:
   void ConnectURLLoaderFactory(
       std::unique_ptr<service_manager::Connector> connector);
 
+  mojom::URLLoaderFactory& GetURLLoaderFactory();
+
   NavigationURLLoaderDelegate* delegate_;
 
-  mojom::URLLoaderFactoryPtr url_loader_factory_;
   mojo::Binding<mojom::URLLoaderClient> binding_;
   mojom::URLLoaderAssociatedPtr url_loader_associated_ptr_;
   scoped_refptr<ResourceResponse> response_;
   SSLStatus ssl_status_;
 
   base::WeakPtrFactory<NavigationURLLoaderNetworkService> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(NavigationURLLoaderNetworkService);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_LOADER_NAVIGATION_URL_LOADER_NETWORK_SERVICE_H_