Stop dispatching encoded-data-length on each data chunk arrival

third_party/WebKit/Source/core/inspector/InspectorNetworkAgent.cpp

Index: third_party/WebKit/Source/core/inspector/InspectorNetworkAgent.cpp
diff --git a/third_party/WebKit/Source/core/inspector/InspectorNetworkAgent.cpp b/third_party/WebKit/Source/core/inspector/InspectorNetworkAgent.cpp
index 89b751538085166dd2f2f59339d20e06ed896149..adfa7df816fa2b15b87b4d70513a0438257b8015 100644
--- a/third_party/WebKit/Source/core/inspector/InspectorNetworkAgent.cpp
+++ b/third_party/WebKit/Source/core/inspector/InspectorNetworkAgent.cpp
@@ -725,7 +725,7 @@ void InspectorNetworkAgent::didReceiveResourceResponse(
   // following didReceiveResponse as there will be no calls to didReceiveData
   // from the network stack.
   if (isNotModified && cachedResource && cachedResource->encodedSize())
-    didReceiveData(frame, identifier, 0, cachedResource->encodedSize(), 0);
+    didReceiveData(frame, identifier, 0, cachedResource->encodedSize());
 }
 
 static bool isErrorStatusCode(int statusCode) {
@@ -735,8 +735,7 @@ static bool isErrorStatusCode(int statusCode) {
 void InspectorNetworkAgent::didReceiveData(LocalFrame*,
                                            unsigned long identifier,
                                            const char* data,
-                                           int dataLength,
-                                           int encodedDataLength) {
+                                           int dataLength) {
   String requestId = IdentifiersFactory::requestId(identifier);
 
   if (data) {
@@ -751,6 +750,14 @@ void InspectorNetworkAgent::didReceiveData(LocalFrame*,
   }
 
   frontend()->dataReceived(requestId, monotonicallyIncreasingTime(), dataLength,
+                           0);
+}
+
+void InspectorNetworkAgent::didReceiveEncodedData(LocalFrame*,
+                                                  unsigned long identifier,
+                                                  int encodedDataLength) {
+  String requestId = IdentifiersFactory::requestId(identifier);
+  frontend()->dataReceived(requestId, monotonicallyIncreasingTime(), 0,

[dgozman, 2016/11/30 18:02:17]

This looks bad: we first send non-zero data with zero encoded length, then we
send zero data with non-zero encoded length. This is very confusing for anyone
receiving the notification. We should instead send them together in the same
notification.

[yhirano, 2016/12/01 10:33:10]

|length| and |encoded_data_length| are different concepts and there is no strong
reason to dispatch them altogether. The values will be dispatched separately
from the browser process with mojo, and there will be no one-to-one relationship
between the messages.

                            encodedDataLength);
 }