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

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 void URLLoaderFactoryImpl::CreateLoaderAndStart( 66 void URLLoaderFactoryImpl::CreateLoaderAndStart(
66 mojom::URLLoaderRequest request, 67 mojom::URLLoaderAssociatedRequest request,
67 int32_t routing_id, 68 int32_t routing_id,
68 int32_t request_id, 69 int32_t request_id,
69 const ResourceRequest& url_request, 70 const ResourceRequest& url_request,
70 mojom::URLLoaderClientPtr client, 71 mojom::URLLoaderClientAssociatedPtrInfo client_ptr_info,
71 ResourceMessageFilter* filter) { 72 ResourceMessageFilter* filter) {
72 DCHECK_CURRENTLY_ON(BrowserThread::IO); 73 DCHECK_CURRENTLY_ON(BrowserThread::IO);
73 74
75 mojom::URLLoaderClientAssociatedPtr client;
76 client.Bind(std::move(client_ptr_info));
77
74 ResourceDispatcherHostImpl* rdh = ResourceDispatcherHostImpl::Get(); 78 ResourceDispatcherHostImpl* rdh = ResourceDispatcherHostImpl::Get();
75 rdh->OnRequestResourceWithMojo(routing_id, request_id, url_request, 79 rdh->OnRequestResourceWithMojo(routing_id, request_id, url_request,
76 std::move(request), std::move(client), filter); 80 std::move(request), std::move(client), filter);
77 } 81 }
78 82
79 void URLLoaderFactoryImpl::SyncLoad(int32_t routing_id, 83 void URLLoaderFactoryImpl::SyncLoad(int32_t routing_id,
80 int32_t request_id, 84 int32_t request_id,
81 const ResourceRequest& url_request, 85 const ResourceRequest& url_request,
82 const SyncLoadCallback& callback, 86 const SyncLoadCallback& callback,
83 ResourceMessageFilter* filter) { 87 ResourceMessageFilter* filter) {
84 DCHECK_CURRENTLY_ON(BrowserThread::IO); 88 DCHECK_CURRENTLY_ON(BrowserThread::IO);
85 89
86 ResourceDispatcherHostImpl* rdh = ResourceDispatcherHostImpl::Get(); 90 ResourceDispatcherHostImpl* rdh = ResourceDispatcherHostImpl::Get();
87 rdh->OnSyncLoadWithMojo(routing_id, request_id, url_request, filter, 91 rdh->OnSyncLoadWithMojo(routing_id, request_id, url_request, filter,
88 base::Bind(&DispatchSyncLoadResult, callback)); 92 base::Bind(&DispatchSyncLoadResult, callback));
89 } 93 }
90 94
91 void URLLoaderFactoryImpl::Create( 95 void URLLoaderFactoryImpl::Create(
92 scoped_refptr<ResourceMessageFilter> filter, 96 scoped_refptr<ResourceMessageFilter> filter,
93 mojo::InterfaceRequest<mojom::URLLoaderFactory> request) { 97 mojo::InterfaceRequest<mojom::URLLoaderFactory> request) {
94 mojo::MakeStrongBinding( 98 mojo::MakeStrongBinding(
95 base::WrapUnique(new URLLoaderFactoryImpl(std::move(filter))), 99 base::WrapUnique(new URLLoaderFactoryImpl(std::move(filter))),
96 std::move(request)); 100 std::move(request));
97 } 101 }
98 102
99 } // namespace content 103 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698