Chromium Code Reviews| Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.cc |
| diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.cc |
| index 9b3a9b6a23e252f9884878b991777c82316accb0..5fd509ff978512f41f6921e82acd13f3b0beeff2 100644 |
| --- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.cc |
| +++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.cc |
| @@ -174,12 +174,11 @@ void DataReductionProxyNetworkDelegate::OnBeforeSendProxyHeadersInternal( |
| net::HttpRequestHeaders* headers) { |
| DCHECK(data_reduction_proxy_config_); |
| - bool is_using_lofi_mode = false; |
| - |
| if (data_reduction_proxy_io_data_ && |
| data_reduction_proxy_io_data_->lofi_decider() && request) { |
| LoFiDecider* lofi_decider = data_reduction_proxy_io_data_->lofi_decider(); |
| - is_using_lofi_mode = lofi_decider->IsUsingLoFiMode(*request); |
| + bool is_using_lofi_mode = |
|
tbansal1
2015/11/23 20:39:52
Seems like this temp variable is not needed or at
megjablon
2015/11/23 20:53:55
The conditional below only runs on mainframe reque
|
| + lofi_decider->MaybeAddLoFiDirectiveToHeaders(*request, headers); |
| if ((request->load_flags() & net::LOAD_MAIN_FRAME) != 0) { |
| // TODO(megjablon): Need to switch to per page. |
| @@ -190,7 +189,7 @@ void DataReductionProxyNetworkDelegate::OnBeforeSendProxyHeadersInternal( |
| if (data_reduction_proxy_request_options_) { |
| data_reduction_proxy_request_options_->MaybeAddRequestHeader( |
| - request, proxy_info.proxy_server(), headers, is_using_lofi_mode); |
| + request, proxy_info.proxy_server(), headers); |
| } |
| } |