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

Unified Diff: webkit/child/weburlloader_impl.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, 4 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: webkit/child/weburlloader_impl.cc
diff --git a/webkit/child/weburlloader_impl.cc b/webkit/child/weburlloader_impl.cc
index 63ce778da190cc1ab6e217469ad82afdef646bb8..7a4f1f68af5294ee4c0a1482d563766fc2afd800 100644
--- a/webkit/child/weburlloader_impl.cc
+++ b/webkit/child/weburlloader_impl.cc
@@ -346,7 +346,7 @@ class WebURLLoaderImpl::Context : public base::RefCounted<Context>,
bool* has_new_first_party_for_cookies,
GURL* new_first_party_for_cookies) OVERRIDE;
virtual void OnReceivedResponse(const ResourceResponseInfo& info) OVERRIDE;
- virtual void OnDownloadedData(int len) OVERRIDE;
+ virtual void OnDownloadedData(int len, int encoded_data_length) OVERRIDE;
virtual void OnReceivedData(const char* data,
int data_length,
int encoded_data_length) OVERRIDE;
@@ -670,9 +670,10 @@ void WebURLLoaderImpl::Context::OnReceivedResponse(
}
}
-void WebURLLoaderImpl::Context::OnDownloadedData(int len) {
+void WebURLLoaderImpl::Context::OnDownloadedData(int len,
+ int encoded_data_length) {
if (client_)
- client_->didDownloadData(loader_, len);
+ client_->didDownloadData(loader_, len, encoded_data_length);
michaeln 2013/08/29 20:19:05 need to do the two-sided patch dance here
yusukesuzuki 2013/08/30 01:13:23 Right. This patch will become green after Blink is
}
void WebURLLoaderImpl::Context::OnReceivedData(const char* data,
« webkit/child/resource_loader_bridge.h ('K') | « webkit/child/resource_loader_bridge.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698