Get rid of URLLoaderFactory in browser-side case

content/common/url_loader.mojom

 // Copyright 2016 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.
 
 module content.mojom;
 
 [Native]
 struct URLRequest;
 
 [Native]
@@ skipping 13 matching lines @@
   kThrottled = 0,
   kIdle,
   kLowest,
   kLow,
   kMedium,
   kHighest
 };
 
 // Destroying a URLLoader will cancel the associated request.
 interface URLLoader {
+  // Starts loading and binds the given |loader| request with this loader.
+  // This method is intended to be called in C++ impl (as it is not
+  // possible to Start this before |loader| binding in mojo layer), and
+  // is not necessary to call this if the loader is created and stgarted by
+  // URLLoaderFactory.
+  // It is not valid to call this method more than once.
+  Start(associated URLLoader& loader, URLLoaderClient client);

[kinuko, 2017/06/05 09:44:23]

Having a method like this in mojom feels a bit weird, while for C++ use cases
this makes things handier.

What do you think? Any alternative suggestions also really welcome.

+
   // If the associated request has |auto_follow_redirects| set to false,
   // then upon receiving an URLResponse with a non-NULL |redirect_url| field,
   // |FollowRedirect| may be called to load the URL indicated by the redirect.
   FollowRedirect();
 
   // Sets the request priority.
   // |intra_priority_value| is a lesser priority which is used to prioritize
   // requests within a given priority level.
   SetPriority(RequestPriority priority, int32 intra_priority_value);
 };
@@ skipping 48 matching lines @@
 
   // Called when the loader starts loading response body. This is called after
   // OnReceiveResponse is called.
   OnStartLoadingResponseBody(handle<data_pipe_consumer> body);
 
   // Called when the loading completes. No notification will be dispatched for
   // this client after this message arrives.
   OnComplete(URLLoaderStatus completion_status);
 };