Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(343)

Unified Diff: content/network/url_loader_impl.h

Issue 2919313004: Get rid of URLLoaderFactory in browser-side case (Closed)
Patch Set: . Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/network/url_loader_impl.h
diff --git a/content/network/url_loader_impl.h b/content/network/url_loader_impl.h
index 98337d9c3db9eba23451f5eb372bdadd121b9511..a8291b870441e8273f0668115b49fad57ad4b359 100644
--- a/content/network/url_loader_impl.h
+++ b/content/network/url_loader_impl.h
@@ -24,17 +24,20 @@ 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;

Powered by Google App Engine
This is Rietveld 408576698