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

Side by Side Diff: content/browser/loader/url_loader_factory_impl.cc

Issue 2449933003: Use Associated interfaces for mojo-loading (Closed)
Patch Set: fix Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/loader/url_loader_factory_impl.h" 5 #include "content/browser/loader/url_loader_factory_impl.h"
6 6
7 #include "content/browser/loader/resource_dispatcher_host_impl.h" 7 #include "content/browser/loader/resource_dispatcher_host_impl.h"
8 #include "content/browser/loader/resource_message_filter.h" 8 #include "content/browser/loader/resource_message_filter.h"
9 #include "content/common/resource_request.h" 9 #include "content/common/resource_request.h"
10 #include "content/common/url_loader.mojom.h" 10 #include "content/common/url_loader.mojom.h"
(...skipping 27 matching lines...) Expand all
38 : resource_message_filter_(std::move(resource_message_filter)) { 38 : resource_message_filter_(std::move(resource_message_filter)) {
39 DCHECK(resource_message_filter_); 39 DCHECK(resource_message_filter_);
40 DCHECK_CURRENTLY_ON(BrowserThread::IO); 40 DCHECK_CURRENTLY_ON(BrowserThread::IO);
41 } 41 }
42 42
43 URLLoaderFactoryImpl::~URLLoaderFactoryImpl() { 43 URLLoaderFactoryImpl::~URLLoaderFactoryImpl() {
44 DCHECK_CURRENTLY_ON(BrowserThread::IO); 44 DCHECK_CURRENTLY_ON(BrowserThread::IO);
45 } 45 }
46 46
47 void URLLoaderFactoryImpl::CreateLoaderAndStart( 47 void URLLoaderFactoryImpl::CreateLoaderAndStart(
48 mojom::URLLoaderRequest request, 48 mojom::URLLoaderAssociatedRequest request,
49 int32_t routing_id, 49 int32_t routing_id,
50 int32_t request_id, 50 int32_t request_id,
51 const ResourceRequest& url_request, 51 const ResourceRequest& url_request,
52 mojom::URLLoaderClientPtr client) { 52 mojom::URLLoaderClientAssociatedPtrInfo client_ptr_info) {
53 CreateLoaderAndStart(std::move(request), routing_id, request_id, url_request, 53 CreateLoaderAndStart(std::move(request), routing_id, request_id, url_request,
54 std::move(client), resource_message_filter_.get()); 54 std::move(client_ptr_info),
55 resource_message_filter_.get());
55 } 56 }
56 57
57 void URLLoaderFactoryImpl::SyncLoad(int32_t routing_id, 58 void URLLoaderFactoryImpl::SyncLoad(int32_t routing_id,
58 int32_t request_id, 59 int32_t request_id,
59 const ResourceRequest& url_request, 60 const ResourceRequest& url_request,
60 const SyncLoadCallback& callback) { 61 const SyncLoadCallback& callback) {
61 SyncLoad(routing_id, request_id, url_request, callback, 62 SyncLoad(routing_id, request_id, url_request, callback,
62 resource_message_filter_.get()); 63 resource_message_filter_.get());
63 } 64 }
64 65
65 // static 66 // static
66 void URLLoaderFactoryImpl::CreateLoaderAndStart( 67 void URLLoaderFactoryImpl::CreateLoaderAndStart(
67 mojom::URLLoaderRequest request, 68 mojom::URLLoaderAssociatedRequest request,
68 int32_t routing_id, 69 int32_t routing_id,
69 int32_t request_id, 70 int32_t request_id,
70 const ResourceRequest& url_request, 71 const ResourceRequest& url_request,
71 mojom::URLLoaderClientPtr client, 72 mojom::URLLoaderClientAssociatedPtrInfo client_ptr_info,
72 ResourceMessageFilter* filter) { 73 ResourceMessageFilter* filter) {
73 DCHECK_CURRENTLY_ON(BrowserThread::IO); 74 DCHECK_CURRENTLY_ON(BrowserThread::IO);
74 75
76 mojom::URLLoaderClientAssociatedPtr client;
77 client.Bind(std::move(client_ptr_info));
78
75 ResourceDispatcherHostImpl* rdh = ResourceDispatcherHostImpl::Get(); 79 ResourceDispatcherHostImpl* rdh = ResourceDispatcherHostImpl::Get();
76 rdh->OnRequestResourceWithMojo(routing_id, request_id, url_request, 80 rdh->OnRequestResourceWithMojo(routing_id, request_id, url_request,
77 std::move(request), std::move(client), filter); 81 std::move(request), std::move(client), filter);
78 } 82 }
79 83
80 // static 84 // static
81 void URLLoaderFactoryImpl::SyncLoad(int32_t routing_id, 85 void URLLoaderFactoryImpl::SyncLoad(int32_t routing_id,
82 int32_t request_id, 86 int32_t request_id,
83 const ResourceRequest& url_request, 87 const ResourceRequest& url_request,
84 const SyncLoadCallback& callback, 88 const SyncLoadCallback& callback,
85 ResourceMessageFilter* filter) { 89 ResourceMessageFilter* filter) {
86 DCHECK_CURRENTLY_ON(BrowserThread::IO); 90 DCHECK_CURRENTLY_ON(BrowserThread::IO);
87 91
88 ResourceDispatcherHostImpl* rdh = ResourceDispatcherHostImpl::Get(); 92 ResourceDispatcherHostImpl* rdh = ResourceDispatcherHostImpl::Get();
89 rdh->OnSyncLoadWithMojo(routing_id, request_id, url_request, filter, 93 rdh->OnSyncLoadWithMojo(routing_id, request_id, url_request, filter,
90 base::Bind(&DispatchSyncLoadResult, callback)); 94 base::Bind(&DispatchSyncLoadResult, callback));
91 } 95 }
92 96
93 void URLLoaderFactoryImpl::Create( 97 void URLLoaderFactoryImpl::Create(
94 scoped_refptr<ResourceMessageFilter> filter, 98 scoped_refptr<ResourceMessageFilter> filter,
95 mojo::InterfaceRequest<mojom::URLLoaderFactory> request) { 99 mojo::InterfaceRequest<mojom::URLLoaderFactory> request) {
96 mojo::MakeStrongBinding( 100 mojo::MakeStrongBinding(
97 base::WrapUnique(new URLLoaderFactoryImpl(std::move(filter))), 101 base::WrapUnique(new URLLoaderFactoryImpl(std::move(filter))),
98 std::move(request)); 102 std::move(request));
99 } 103 }
100 104
101 } // namespace content 105 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/loader/url_loader_factory_impl.h ('k') | content/browser/loader/url_loader_factory_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698