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

Unified Diff: content/child/resource_dispatcher.cc

Issue 2574143003: Implement upload progress handling in Mojo loading (Closed)
Patch Set: +comment Created 3 years, 11 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/resource_dispatcher.cc
diff --git a/content/child/resource_dispatcher.cc b/content/child/resource_dispatcher.cc
index 38efe244a9a2e2f1ed58b6aa3dddfe07d929186b..09c7db9058e1b113afaf00134802bd9c250c9251 100644
--- a/content/child/resource_dispatcher.cc
+++ b/content/child/resource_dispatcher.cc
@@ -157,8 +157,12 @@ void ResourceDispatcher::OnUploadProgress(int request_id,
request_info->peer->OnUploadProgress(position, size);
- // Acknowledge receipt
- message_sender_->Send(new ResourceHostMsg_UploadProgress_ACK(request_id));
+ // URLLoaderClientImpl has its own acknowledgement, and doesn't need the IPC
+ // message here.
+ if (!request_info->url_loader) {
+ // Acknowledge receipt
+ message_sender_->Send(new ResourceHostMsg_UploadProgress_ACK(request_id));
+ }
}
void ResourceDispatcher::OnReceivedResponse(
« no previous file with comments | « content/browser/service_worker/service_worker_fetch_dispatcher.cc ('k') | content/child/url_loader_client_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698