Dispatch encoded_data_length separately in content/child

content/public/child/request_peer.h

Index: content/public/child/request_peer.h
diff --git a/content/public/child/request_peer.h b/content/public/child/request_peer.h
index 3390dbea189350317547029ef4067b03954a8d92..7e6ee33e9ab7d9b31a15770e38730eb93745e454 100644
--- a/content/public/child/request_peer.h
+++ b/content/public/child/request_peer.h
@@ -47,12 +47,6 @@ class CONTENT_EXPORT RequestPeer {
     virtual ~ReceivedData() {}
     virtual const char* payload() const = 0;
     virtual int length() const = 0;
-    // The encoded_data_length is the length of the encoded data transferred
-    // over the network, including headers. It is only set for responses
-    // originating from the network (ie. not the cache). It will usually be
-    // different from length(), and may be smaller if the content was
-    // compressed. -1 means this value is unavailable.
-    virtual int encoded_data_length() const = 0;
   };
 
   // A ThreadSafeReceivedData can be deleted on ANY thread.
@@ -86,6 +80,13 @@ class CONTENT_EXPORT RequestPeer {
   // be called multiple times or not at all if an error occurs.
   virtual void OnReceivedData(std::unique_ptr<ReceivedData> data) = 0;
 
+  // Called when the transfer size is updated. This method may be called
+  // multiple times or not at all. The transfer size is the length of the
+  // response (including both headers and the body) over the network.
+  // |transfer_size_diff| is the difference from the value previously reported
+  // one (including the one in OnReceivedResponse). It must be positive.
+  virtual void OnTransferSizeUpdated(int transfer_size_diff) = 0;

[kinuko, 2016/12/07 04:06:26]

Should transfer_size_diff be unsigned if it must always be positive?

[yhirano, 2016/12/07 12:06:58]

Google c++ style guide says 

"
You should not use the unsigned integer types such as uint32_t, unless there is
a valid reason such as representing a bit pattern rather than a number, or you
need defined overflow modulo 2^N. In particular, do not use unsigned types to
say a number will never be negative. Instead, use assertions for this.
"

https://google.github.io/styleguide/cppguide.html#Integer_Types

+
   // Called when metadata generated by the renderer is retrieved from the
   // cache. This method may be called zero or one times.
   virtual void OnReceivedCachedMetadata(const char* data, int len) {}