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

Unified Diff: content/browser/loader/mojo_async_resource_handler_unittest.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 side-by-side diff with in-line comments
Download patch
Index: content/browser/loader/mojo_async_resource_handler_unittest.cc
diff --git a/content/browser/loader/mojo_async_resource_handler_unittest.cc b/content/browser/loader/mojo_async_resource_handler_unittest.cc
index 616f30482b5173d7b377545b4e424c2d2c7a51b9..222bf3db99be428b7b8f72413a514bed45885e78 100644
--- a/content/browser/loader/mojo_async_resource_handler_unittest.cc
+++ b/content/browser/loader/mojo_async_resource_handler_unittest.cc
@@ -223,8 +223,8 @@ class MojoAsyncResourceHandlerWithCustomDataPipeOperations
MojoAsyncResourceHandlerWithCustomDataPipeOperations(
net::URLRequest* request,
ResourceDispatcherHostImpl* rdh,
- mojo::InterfaceRequest<mojom::URLLoader> mojo_request,
- mojom::URLLoaderClientPtr url_loader_client)
+ mojom::URLLoaderAssociatedRequest mojo_request,
+ mojom::URLLoaderClientAssociatedPtr url_loader_client)
: MojoAsyncResourceHandler(request,
rdh,
std::move(mojo_request),
@@ -290,9 +290,12 @@ class MojoAsyncResourceHandlerTestBase {
true, // is_async
false // is_using_lofi
);
+ mojom::URLLoaderClientAssociatedPtrInfo client_ptr_info =
+ url_loader_client_.CreateLocalAssociatedPtrInfo();
+ mojom::URLLoaderClientAssociatedPtr client_ptr;
+ client_ptr.Bind(std::move(client_ptr_info));
handler_.reset(new MojoAsyncResourceHandlerWithCustomDataPipeOperations(
- request_.get(), &rdh_, nullptr,
- url_loader_client_.CreateInterfacePtrAndBind()));
+ request_.get(), &rdh_, nullptr, std::move(client_ptr)));
handler_->SetController(&resource_controller_);
}
« no previous file with comments | « content/browser/loader/mojo_async_resource_handler.cc ('k') | content/browser/loader/resource_dispatcher_host_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698