diff options
author | tzik <tzik@chromium.org> | 2016-01-27 23:13:18 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-01-28 07:14:49 +0000 |
commit | 75df24fa475917adfff5604b7b5b040ffd0d0e19 (patch) | |
tree | 8ef49131a4776e2de1812dd9db6db3ca4aef242a | |
parent | 6d2d7c3b60d3e5cd9b2efd7120cae737ad1e4329 (diff) | |
download | chromium_src-75df24fa475917adfff5604b7b5b040ffd0d0e19.zip chromium_src-75df24fa475917adfff5604b7b5b040ffd0d0e19.tar.gz chromium_src-75df24fa475917adfff5604b7b5b040ffd0d0e19.tar.bz2 |
Factor out encoded length calculation in AsyncResourceHandler
BUG=580928
Review URL: https://codereview.chromium.org/1626213002
Cr-Commit-Position: refs/heads/master@{#372041}
-rw-r--r-- | content/browser/loader/async_resource_handler.cc | 16 | ||||
-rw-r--r-- | content/browser/loader/async_resource_handler.h | 1 |
2 files changed, 10 insertions, 7 deletions
diff --git a/content/browser/loader/async_resource_handler.cc b/content/browser/loader/async_resource_handler.cc index 53b26e1..e6b1ddf 100644 --- a/content/browser/loader/async_resource_handler.cc +++ b/content/browser/loader/async_resource_handler.cc @@ -331,10 +331,7 @@ bool AsyncResourceHandler::OnReadCompleted(int bytes_read, bool* defer) { } int data_offset = buffer_->GetLastAllocationOffset(); - - int64_t current_transfer_size = request()->GetTotalReceivedBytes(); - int encoded_data_length = current_transfer_size - reported_transfer_size_; - reported_transfer_size_ = current_transfer_size; + int encoded_data_length = CalculateEncodedDataLengthToReport(); // TODO(erikchen): Temporary debugging. http://crbug.com/527588. CHECK_LE(data_offset, kBufferSize); @@ -355,9 +352,7 @@ bool AsyncResourceHandler::OnReadCompleted(int bytes_read, bool* defer) { } void AsyncResourceHandler::OnDataDownloaded(int bytes_downloaded) { - int64_t current_transfer_size = request()->GetTotalReceivedBytes(); - int encoded_data_length = current_transfer_size - reported_transfer_size_; - reported_transfer_size_ = current_transfer_size; + int encoded_data_length = CalculateEncodedDataLengthToReport(); ResourceMessageFilter* filter = GetFilter(); if (filter) { @@ -455,4 +450,11 @@ bool AsyncResourceHandler::CheckForSufficientResource() { return false; } +int AsyncResourceHandler::CalculateEncodedDataLengthToReport() { + int64_t current_transfer_size = request()->GetTotalReceivedBytes(); + int encoded_data_length = current_transfer_size - reported_transfer_size_; + reported_transfer_size_ = current_transfer_size; + return encoded_data_length; +} + } // namespace content diff --git a/content/browser/loader/async_resource_handler.h b/content/browser/loader/async_resource_handler.h index efc597f..ca06ed3 100644 --- a/content/browser/loader/async_resource_handler.h +++ b/content/browser/loader/async_resource_handler.h @@ -66,6 +66,7 @@ class AsyncResourceHandler : public ResourceHandler, void ResumeIfDeferred(); void OnDefer(); bool CheckForSufficientResource(); + int CalculateEncodedDataLengthToReport(); scoped_refptr<ResourceBuffer> buffer_; ResourceDispatcherHostImpl* rdh_; |