Propagate the encoded data length through OnDownloadData delegate method

webkit/child/resource_loader_bridge.h

 // 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.
 //
 // The intent of this file is to provide a type-neutral abstraction between
 // Chrome and WebKit for resource loading. This pure-virtual interface is
 // implemented by the embedder.
 //
 // One of these objects will be created by WebKit for each request. WebKit
 // will own the pointer to the bridge, and will delete it when the request is
@@ skipping 135 matching lines @@
                                     GURL* new_first_party_for_cookies) = 0;
 
     // Called when response headers are available (after all redirects have
     // been followed).
     virtual void OnReceivedResponse(const ResourceResponseInfo& info) = 0;
 
     // Called when a chunk of response data is downloaded.  This method may be
     // called multiple times or not at all if an error occurs.  This method is
     // only called if RequestInfo::download_to_file was set to true, and in
     // that case, OnReceivedData will not be called.
-    virtual void OnDownloadedData(int len) = 0;
+    // The encoded_data_length is the length of the encoded data transferred
+    // over the network, which could be different from data length (e.g. for
+    // gzipped content), or -1 if if unknown.

[Nico, 2013/09/02 03:50:25]

s/if if/if/

So this will only be valid while devtools are open? If so, and that's
intentional, mention that in the CL description which why that is sufficient.
With that, lgtm.

[yusukesuzuki, 2013/09/02 04:10:14]

Done. I've mentioned it in the CL description and documented it in the comment.
And I've done s/if if/if/. Thanks.

+    virtual void OnDownloadedData(int len, int encoded_data_length) = 0;
 
     // Called when a chunk of response data is available. This method may
     // be called multiple times or not at all if an error occurs.
     // The encoded_data_length is the length of the encoded data transferred
     // over the network, which could be different from data length (e.g. for
     // gzipped content), or -1 if if unknown.
     virtual void OnReceivedData(const char* data,
                                 int data_length,
                                 int encoded_data_length) = 0;
 
@@ skipping 56 matching lines @@
   // methods may be called to construct the body of the request.
   WEBKIT_CHILD_EXPORT ResourceLoaderBridge();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ResourceLoaderBridge);
 };
 
 }  // namespace webkit_glue
 
 #endif  // WEBKIT_CHILD_RESOURCE_LOADER_BRIDGE_H_