Render process changes for ResourceTiming sizes.

third_party/WebKit/public/platform/WebURLLoaderClient.h

Index: third_party/WebKit/public/platform/WebURLLoaderClient.h
diff --git a/third_party/WebKit/public/platform/WebURLLoaderClient.h b/third_party/WebKit/public/platform/WebURLLoaderClient.h
index 41b7bee7db94cf0f07c1b382f38b047b64651e6c..a74e9c5216a8ccc4e0c3ba321fe02e7be3c790fb 100644
--- a/third_party/WebKit/public/platform/WebURLLoaderClient.h
+++ b/third_party/WebKit/public/platform/WebURLLoaderClient.h
@@ -46,7 +46,7 @@ public:
     // Called when following a redirect. |newRequest| contains the request
     // generated by the redirect. The client may modify |newRequest|.

[kinuko, 2016/07/04 04:08:12]

could we update the comment?

[Adam Rice, 2016/07/04 06:33:42]

Done.

     virtual void willFollowRedirect(
-        WebURLLoader*, WebURLRequest& newRequest, const WebURLResponse& redirectResponse) { }
+        WebURLLoader*, WebURLRequest& newRequest, const WebURLResponse& redirectResponse, int64_t encodedDataLength) {}
 
     // Called to report upload progress. The bytes reported correspond to
     // the HTTP message body.
@@ -69,7 +69,7 @@ public:
     virtual void didDownloadData(WebURLLoader*, int dataLength, int encodedDataLength) { }
 
     // Called when a chunk of response data is received.

[kinuko, 2016/07/04 04:08:12]

Ditto.

Also: similar to other comments, we seem to often pass in -1 or 0 as
encodedDataLength when it's unknown or not applicable... is it possible to be
consistent here and (or at least) explicitly comment what should be passed in in
such cases?

(Contrary to this one for willFollowRedirect we basically seem to pass in 0 in
various tests, which feels also confusing to me and would be better if it's
documented)

[Adam Rice, 2016/07/04 06:33:42]

I have documented this method as taking -1 when the information is unavailable.
For the same reason as with ReceivedData, I think this is undesirable and I
would like to change it.

-    virtual void didReceiveData(WebURLLoader*, const char* data, int dataLength, int encodedDataLength) { }
+    virtual void didReceiveData(WebURLLoader*, const char* data, int dataLength, int encodedDataLength, int encodedBodyLength) {}
 
     // Called when a chunk of renderer-generated metadata is received from the cache.
     virtual void didReceiveCachedMetadata(WebURLLoader*, const char* data, int dataLength) { }