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

Unified Diff: content/child/url_loader_client_impl.cc

Issue 2715423003: [Mojo-Loading] Use independent URLLoaderClient (Closed)
Patch Set: fix Created 3 years, 10 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/child/url_loader_client_impl.cc
diff --git a/content/child/url_loader_client_impl.cc b/content/child/url_loader_client_impl.cc
index 0ade2475acf9b8d052fbf54f2c23c2103a83df22..e0d8c5387569c681b31e0815c96ce0a3295f084d 100644
--- a/content/child/url_loader_client_impl.cc
+++ b/content/child/url_loader_client_impl.cc
@@ -28,11 +28,8 @@ URLLoaderClientImpl::~URLLoaderClientImpl() {
body_consumer_->Cancel();
}
-void URLLoaderClientImpl::Bind(
- mojom::URLLoaderClientAssociatedPtrInfo* client_ptr_info) {
- // TODO(yhirano): Use |task_runner_| here.
- // Currently it is unable because of a ChannelAssociatedInterface restriction.
- binding_.Bind(client_ptr_info);
+void URLLoaderClientImpl::Bind(mojom::URLLoaderClientPtr* client_ptr) {
+ binding_.Bind(client_ptr, task_runner_);
}
void URLLoaderClientImpl::SetDefersLoading() {
@@ -112,9 +109,9 @@ void URLLoaderClientImpl::FlushDeferredMessages() {
void URLLoaderClientImpl::OnReceiveResponse(
const ResourceResponseHead& response_head,
- mojom::DownloadedTempFileAssociatedPtrInfo downloaded_file) {
+ mojom::DownloadedTempFilePtr downloaded_file) {
has_received_response_ = true;
- downloaded_file_.Bind(std::move(downloaded_file));
+ downloaded_file_ = std::move(downloaded_file);
Dispatch(ResourceMsg_ReceivedResponse(request_id_, response_head));
}

Powered by Google App Engine
This is Rietveld 408576698