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

Side by Side Diff: content/renderer/pepper/pepper_url_loader_host.cc

Issue 23696002: Propagate the encoded data length through OnDownloadData delegate method (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 3 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 unified diff | Download patch
« no previous file with comments | « content/renderer/pepper/pepper_url_loader_host.h ('k') | webkit/child/resource_loader_bridge.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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/renderer/pepper/pepper_url_loader_host.h" 5 #include "content/renderer/pepper/pepper_url_loader_host.h"
6 6
7 #include "content/renderer/pepper/pepper_plugin_instance_impl.h" 7 #include "content/renderer/pepper/pepper_plugin_instance_impl.h"
8 #include "content/renderer/pepper/renderer_ppapi_host_impl.h" 8 #include "content/renderer/pepper/renderer_ppapi_host_impl.h"
9 #include "content/renderer/pepper/url_request_info_util.h" 9 #include "content/renderer/pepper/url_request_info_util.h"
10 #include "content/renderer/pepper/url_response_info_util.h" 10 #include "content/renderer/pepper/url_response_info_util.h"
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
145 void PepperURLLoaderHost::didReceiveResponse(WebURLLoader* loader, 145 void PepperURLLoaderHost::didReceiveResponse(WebURLLoader* loader,
146 const WebURLResponse& response) { 146 const WebURLResponse& response) {
147 // Sets -1 if the content length is unknown. Send before issuing callback. 147 // Sets -1 if the content length is unknown. Send before issuing callback.
148 total_bytes_to_be_received_ = response.expectedContentLength(); 148 total_bytes_to_be_received_ = response.expectedContentLength();
149 UpdateProgress(); 149 UpdateProgress();
150 150
151 SaveResponse(response); 151 SaveResponse(response);
152 } 152 }
153 153
154 void PepperURLLoaderHost::didDownloadData(WebURLLoader* loader, 154 void PepperURLLoaderHost::didDownloadData(WebURLLoader* loader,
155 int data_length) { 155 int data_length,
156 int encoded_data_length) {
156 bytes_received_ += data_length; 157 bytes_received_ += data_length;
157 UpdateProgress(); 158 UpdateProgress();
158 } 159 }
159 160
160 void PepperURLLoaderHost::didReceiveData(WebURLLoader* loader, 161 void PepperURLLoaderHost::didReceiveData(WebURLLoader* loader,
161 const char* data, 162 const char* data,
162 int data_length, 163 int data_length,
163 int encoded_data_length) { 164 int encoded_data_length) {
164 // Note that |loader| will be NULL for document loads. 165 // Note that |loader| will be NULL for document loads.
165 bytes_received_ += data_length; 166 bytes_received_ += data_length;
(...skipping 247 matching lines...) Expand 10 before | Expand all | Expand 10 after
413 ppapi::proxy::ResourceMessageReplyParams params; 414 ppapi::proxy::ResourceMessageReplyParams params;
414 SendUpdateToPlugin(new PpapiPluginMsg_URLLoader_UpdateProgress( 415 SendUpdateToPlugin(new PpapiPluginMsg_URLLoader_UpdateProgress(
415 record_upload ? bytes_sent_ : -1, 416 record_upload ? bytes_sent_ : -1,
416 record_upload ? total_bytes_to_be_sent_ : -1, 417 record_upload ? total_bytes_to_be_sent_ : -1,
417 record_download ? bytes_received_ : -1, 418 record_download ? bytes_received_ : -1,
418 record_download ? total_bytes_to_be_received_ : -1)); 419 record_download ? total_bytes_to_be_received_ : -1));
419 } 420 }
420 } 421 }
421 422
422 } // namespace content 423 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/pepper/pepper_url_loader_host.h ('k') | webkit/child/resource_loader_bridge.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698