Fixing DataReductionProxy.BypassedBytes.LocalBypassRules

chrome/browser/net/chrome_network_delegate.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 "chrome/browser/net/chrome_network_delegate.h"
 
 #include <stdlib.h>
 
 #include <vector>
 
@@ skipping 565 matching lines @@
       int64 adjusted_original_content_length =
           data_reduction_proxy::GetAdjustedOriginalContentLength(
               request_type, original_content_length,
               received_content_length);
       AccumulateContentLength(received_content_length,
                               adjusted_original_content_length,
                               request_type);
       RecordContentLengthHistograms(received_content_length,
                                     original_content_length,
                                     freshness_lifetime);
-      if (data_reduction_proxy_enabled_ && data_reduction_proxy_usage_stats_) {
+      if (data_reduction_proxy_enabled_ && data_reduction_proxy_usage_stats_ &&

[bengr, 2014/08/29 19:29:47]

nit: It might be a bit cleaner to put each condition on its own line

[megjablon, 2014/08/29 20:26:32]

Done.

+          !proxy_config_getter_.is_null()) {
         data_reduction_proxy_usage_stats_->RecordBypassedBytesHistograms(
-            *request, *data_reduction_proxy_enabled_);
+            *request,
+            *data_reduction_proxy_enabled_,
+            proxy_config_getter_.Run());
       }
       DVLOG(2) << __FUNCTION__
           << " received content length: " << received_content_length
           << " original content length: " << original_content_length
           << " url: " << request->url();
     }
 
     extensions_delegate_->OnCompleted(request, started);
   } else if (request->status().status() == net::URLRequestStatus::FAILED ||
              request->status().status() == net::URLRequestStatus::CANCELED) {
@@ skipping 218 matching lines @@
     data_reduction_proxy::DataReductionProxyRequestType request_type) {
   DCHECK_GE(received_content_length, 0);
   DCHECK_GE(original_content_length, 0);
   StoreAccumulatedContentLength(received_content_length,
                                 original_content_length,
                                 request_type,
                                 reinterpret_cast<Profile*>(profile_));
   received_content_length_ += received_content_length;
   original_content_length_ += original_content_length;
 }