Render process changes for ResourceTiming sizes.

third_party/WebKit/public/platform/WebURLLoaderClient.h

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 26 matching lines @@
 namespace blink {
 
 class WebURLLoader;
 class WebURLRequest;
 class WebURLResponse;
 struct WebURLError;
 
 class BLINK_PLATFORM_EXPORT WebURLLoaderClient {
 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.
     virtual void didSendData(
         WebURLLoader*, unsigned long long bytesSent, unsigned long long totalBytesToBeSent) { }
 
     // Called when response headers are received.
     virtual void didReceiveResponse(WebURLLoader*, const WebURLResponse&) { }
 
     // Called when response headers are received.
     // The ownership of |handle| is transferred to the callee.
     virtual void didReceiveResponse(WebURLLoader* loader, const WebURLResponse& response, WebDataConsumerHandle* handle)
     {
         delete handle;
         didReceiveResponse(loader, response);
     }
 
     // Called when a chunk of response data is downloaded. This is only called
     // if WebURLRequest's downloadToFile flag was set to true.
     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) { }
 
     // Called when the load completes successfully.
     // |totalEncodedDataLength| may be equal to kUnknownEncodedDataLength.
     virtual void didFinishLoading(WebURLLoader* loader, double finishTime, int64_t totalEncodedDataLength) { }
 
     // Called when the load completes with an error.
     virtual void didFail(WebURLLoader*, const WebURLError&) { }
 
     // Value passed to didFinishLoading when total encoded data length isn't known.
     static const int64_t kUnknownEncodedDataLength = -1;
 
 protected:
     virtual ~WebURLLoaderClient() { }
 };
 
 } // namespace blink
 
 #endif