Propagate the encoded data length through OnDownloadData delegate method

webkit/child/weburlloader_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // An implementation of WebURLLoader in terms of ResourceLoaderBridge.
 
 #include "webkit/child/weburlloader_impl.h"
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
@@ skipping 328 matching lines @@
       WebKitPlatformSupportImpl* platform);
 
   // ResourceLoaderBridge::Peer methods:
   virtual void OnUploadProgress(uint64 position, uint64 size) OVERRIDE;
   virtual bool OnReceivedRedirect(
       const GURL& new_url,
       const ResourceResponseInfo& info,
       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;
   virtual void OnReceivedCachedMetadata(const char* data, int len) OVERRIDE;
   virtual void OnCompletedRequest(
       int error_code,
       bool was_ignored_by_handler,
       const std::string& security_info,
       const base::TimeTicks& completion_time) OVERRIDE;
 
@@ skipping 303 matching lines @@
       multipart_delegate_.reset(
           new MultipartResponseDelegate(client_, loader_, response, boundary));
     }
   } else if (info.mime_type == "text/vnd.chromium.ftp-dir" &&
              !show_raw_listing) {
     ftp_listing_delegate_.reset(
         new FtpDirectoryListingResponseDelegate(client_, loader_, response));
   }
 }
 
-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 rebaselined[1].
After this patch is landed, I'll remove the stub code in Blink allowing 2
arguments style didDownloadData not to break the build.

[1]: https://chromiumcodereview.appspot.com/23632004/

 }
 
 void WebURLLoaderImpl::Context::OnReceivedData(const char* data,
                                                int data_length,
                                                int encoded_data_length) {
   if (!client_)
     return;
 
   if (ftp_listing_delegate_) {
     // The FTP listing delegate will make the appropriate calls to
@@ skipping 158 matching lines @@
 
 void WebURLLoaderImpl::setDefersLoading(bool value) {
   context_->SetDefersLoading(value);
 }
 
 void WebURLLoaderImpl::didChangePriority(WebURLRequest::Priority new_priority) {
   context_->DidChangePriority(new_priority);
 }
 
 }  // namespace webkit_glue