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

Unified Diff: content/child/resource_dispatcher.cc

Issue 2574143003: Implement upload progress handling in Mojo loading (Closed)
Patch Set: update TestExpectations 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 24afd8a80414438645b5c575a23c3b166f4ab70b..e3a7aa5a395ec6b7cb289f0d0dd5b91c1a6bdaa5 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(

Powered by Google App Engine
This is Rietveld 408576698