Propagate the encoded data length through OnDownloadData delegate method

content/browser/loader/async_resource_handler.cc

 // Copyright (c) 2012 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.
 
 #include "content/browser/loader/async_resource_handler.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/command_line.h"
@@ skipping 266 matching lines @@
         "Net.AsyncResourceHandler_PendingDataCount_WhenFull",
         pending_data_count_, 0, 100, 100);
     *defer = did_defer_ = true;
   }
 
   return true;
 }
 
 void AsyncResourceHandler::OnDataDownloaded(
     int request_id, int bytes_downloaded) {
+  int encoded_data_length =
+      DevToolsNetLogObserver::GetAndResetEncodedDataLength(request_);

[michaeln, 2013/08/29 20:19:05]

I'm not familiar with the inspector instrumentation but i see the exact same
handling for the non-download-to-file case in OnReadCompleted so i trust it
returns valid results in this case too since this step replace that step in the
d-t-f control flow.

[Nico, 2013/08/31 20:49:47]

I'm also curious about
this.DevToolsNetLogObserver::GetAndResetEncodedDataLength() seems to return -1
if dev_tools_net_log_observer == NULL, and instance_ in
content/browser/devtools/devtools_netlog_observer.cc seems to be only non-NULL
after Attach() was called (and before Detach() was called), which after skimming
seems to be only the case while devtools is active (?)

[yusukesuzuki, 2013/09/02 02:05:10]

Right. The encoded_data_length is used for the Inspector, so it becomes valid
while devtools is active.
The encoded_data_length may become -1 and it represents that the encoded data
length is unknown.

+
   filter_->Send(new ResourceMsg_DataDownloaded(
-      routing_id_, request_id, bytes_downloaded));
+      routing_id_, request_id, bytes_downloaded, encoded_data_length));
 }
 
 bool AsyncResourceHandler::OnResponseCompleted(
     int request_id,
     const net::URLRequestStatus& status,
     const std::string& security_info) {
   // If we crash here, figure out what URL the renderer was requesting.
   // http://crbug.com/107692
   char url_buf[128];
   base::strlcpy(url_buf, request_->url().spec().c_str(), arraysize(url_buf));
@@ skipping 57 matching lines @@
 }
 
 void AsyncResourceHandler::ResumeIfDeferred() {
   if (did_defer_) {
     did_defer_ = false;
     controller()->Resume();
   }
 }
 
 }  // namespace content