Browser process changes for Resource Timing sizes.

content/child/resource_dispatcher_unittest.cc

Index: content/child/resource_dispatcher_unittest.cc
diff --git a/content/child/resource_dispatcher_unittest.cc b/content/child/resource_dispatcher_unittest.cc
index 07fbecfbb4d8108ebffd283dc04265f4245e3464..4fb722c486e26949aa14de54fc2118b3ab622fe4 100644
--- a/content/child/resource_dispatcher_unittest.cc
+++ b/content/child/resource_dispatcher_unittest.cc
@@ -275,8 +275,8 @@ class ResourceDispatcherTest : public testing::Test, public IPC::Sender {
     memcpy(shared_memory_map_[request_id]->memory(), data.c_str(),
            data.length());
 
-    EXPECT_TRUE(dispatcher_->OnMessageReceived(
-        ResourceMsg_DataReceived(request_id, 0, data.length(), data.length())));
+    EXPECT_TRUE(dispatcher_->OnMessageReceived(ResourceMsg_DataReceived(
+        request_id, 0, data.length(), data.length(), data.length())));

[Mike West, 2016/06/24 10:49:20]

It surprises me that this is the only test change. Is it possible to write some
tests that verify that the correct information is being handed down? And that
the (trivial) change to the implementation of
`CalculateEncodedDataLengthToReport` didn't change the result?

[Adam Rice, 2016/07/07 06:50:09]

AsyncResourceHandler had no tests. I have added some.

I also added a test for the ResourceMsg_InlinedDataChunkReceived IPC to
ResourceDispatcher.

   }
 
   void NotifyDataDownloaded(int request_id, int decoded_length,