Render process changes for ResourceTiming sizes.

content/public/child/request_peer.h

 // Copyright 2014 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.
 
 #ifndef CONTENT_PUBLIC_CHILD_REQUEST_PEER_H_
 #define CONTENT_PUBLIC_CHILD_REQUEST_PEER_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 22 matching lines @@
 // These callbacks mirror net::URLRequest::Delegate and the order and
 // conditions in which they will be called are identical. See url_request.h
 // for more information.
 class CONTENT_EXPORT RequestPeer {
  public:
   // This class represents data gotten from the Browser process. Each data
   // consists of |payload|, |length| and |encoded_length|. The payload is
   // valid only when the data instance is valid.
   // In order to work with Chrome resource loading IPC, it is desirable to
   // reclaim data in FIFO order in a RequestPeer in terms of performance.
   // |payload|, |length| and |encoded_length| functions are thread-safe, but

[kinuko, 2016/06/28 15:35:07]

please update this comment too

[Adam Rice, 2016/06/29 09:52:15]

Done.

   // the data object itself must be destroyed on the original thread.
   class CONTENT_EXPORT ReceivedData {
    public:
     virtual ~ReceivedData() {}
     virtual const char* payload() const = 0;
     virtual int length() const = 0;
     // The encoded_length is the length of the encoded data transferred
     // over the network, which could be different from data length (e.g. for
     // gzipped content).
     virtual int encoded_length() const = 0;
+    // The encoded_body_length is the size of the body as transferred over the
+    // network or stored in the disk cache. This will be different from the data
+    // length if a content encoding was used.
+    virtual int encoded_body_length() const = 0;

[kinuko, 2016/06/28 15:35:07]

It'd be probably nicer to clearly explain how encoded_length and
encoded_body_length differ (rather than the diff from the data length)

(Also... if possible we might want to update 'encoded_length' field name to
encoded_data_length together now?)

[Adam Rice, 2016/06/29 09:52:15]

I have tried to make the comments clearer. I also renamed encoded_length to
encoded_data_length for consistency.

   };
 
   // A ThreadSafeReceivedData can be deleted on ANY thread.
   class CONTENT_EXPORT ThreadSafeReceivedData : public ReceivedData {};
 
   // Called as upload progress is made.
   // note: only for requests with upload progress enabled.
   virtual void OnUploadProgress(uint64_t position, uint64_t size) = 0;
 
   // Called when a redirect occurs.  The implementation may return false to
@@ skipping 32 matching lines @@
                                   const std::string& security_info,
                                   const base::TimeTicks& completion_time,
                                   int64_t total_transfer_size) = 0;
 
   virtual ~RequestPeer() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_CHILD_REQUEST_PEER_H_