Get rid of URLLoaderFactory in browser-side case

content/network/url_loader_impl.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_NETWORK_URL_LOADER_IMPL_H_
 #define CONTENT_NETWORK_URL_LOADER_IMPL_H_
 
 #include <stdint.h>
 
 #include <memory>
 
 #include "base/memory/weak_ptr.h"
 #include "content/common/content_export.h"
 #include "content/common/url_loader.mojom.h"
 #include "mojo/public/cpp/bindings/associated_binding.h"
 #include "mojo/public/cpp/system/simple_watcher.h"
 #include "net/url_request/url_request.h"
 
 namespace content {
 
 class NetworkContext;
 class NetToMojoPendingBuffer;
 
 class CONTENT_EXPORT URLLoaderImpl : public mojom::URLLoader,
                                      public net::URLRequest::Delegate {
  public:
+  // Constructs URLLoaderImpl. The created instance must be owned by the
+  // caller until Start() method is called. Once Start() is called this
+  // self-destructs itself on connection close.
   URLLoaderImpl(NetworkContext* context,
-                mojom::URLLoaderAssociatedRequest url_loader_request,
                 int32_t options,
-                const ResourceRequest& request,
-                mojom::URLLoaderClientPtr url_loader_client);
+                const ResourceRequest& request);
   ~URLLoaderImpl() override;
 
   // Called when the associated NetworkContext is going away.
   void Cleanup();
 
   // mojom::URLLoader implementation:
+  void Start(mojom::URLLoaderAssociatedRequest url_loader_request,
+             mojom::URLLoaderClientPtr url_loader_client) override;
   void FollowRedirect() override;
   void SetPriority(net::RequestPriority priority,
                    int32_t intra_priority_value) override;
 
   // net::URLRequest::Delegate methods:
   void OnReceivedRedirect(net::URLRequest* url_request,
                           const net::RedirectInfo& redirect_info,
                           bool* defer_redirect) override;
   void OnResponseStarted(net::URLRequest* url_request, int net_error) override;
   void OnReadCompleted(net::URLRequest* url_request, int bytes_read) override;
@@ skipping 22 matching lines @@
   mojo::SimpleWatcher peer_closed_handle_watcher_;
 
   base::WeakPtrFactory<URLLoaderImpl> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(URLLoaderImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_NETWORK_URL_LOADER_IMPL_H_