Bypassed Bytes UMAs

chrome/browser/net/chrome_network_delegate.h

 // 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.
 
 #ifndef CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_
 #define CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 104 matching lines @@
 
   void set_enable_do_not_track(BooleanPrefMember* enable_do_not_track) {
     enable_do_not_track_ = enable_do_not_track;
   }
 
   void set_force_google_safe_search(
       BooleanPrefMember* force_google_safe_search) {
     force_google_safe_search_ = force_google_safe_search;
   }
 
+  void set_data_reduction_proxy_enabled_pref(
+      BooleanPrefMember* data_reduction_proxy_enabled) {
+    data_reduction_proxy_enabled_ = data_reduction_proxy_enabled;
+  }

[mmenke, 2014/07/24 19:32:05]

nit:  Setters for all DRP settings should be together, in the same order they're
declared at the bottom of the file (And should probably be together and
initialized in the same order in profil_io_data, unless there's a reason not to
do that).

[megjablon, 2014/07/24 20:09:37]

There's no reason not to do it that way, but right now we have the setters for
all BooleanPrefMembers grouped together. I can change the ordering in
profile_io_data if you like to the same order as the setters here. 

[mmenke, 2014/07/24 20:34:12]

Makes sense.  For some reason, I was thinking that it was declared near the
other DRP settings, but the setter was in a different order (Which is not
actually true).

+
   void set_domain_reliability_monitor(
       domain_reliability::DomainReliabilityMonitor* monitor) {
     domain_reliability_monitor_ = monitor;
   }
 
   void set_prerender_tracker(prerender::PrerenderTracker* prerender_tracker) {
     prerender_tracker_ = prerender_tracker;
   }
 
   // |data_reduction_proxy_params_| must outlive this ChromeNetworkDelegate.
@@ skipping 113 matching lines @@
   void* profile_;
   base::FilePath profile_path_;
   scoped_refptr<CookieSettings> cookie_settings_;
 
   scoped_ptr<chrome_browser_net::ConnectInterceptor> connect_interceptor_;
 
   // Weak, owned by our owner.
   BooleanPrefMember* enable_referrers_;
   BooleanPrefMember* enable_do_not_track_;
   BooleanPrefMember* force_google_safe_search_;
+  BooleanPrefMember* data_reduction_proxy_enabled_;
 
   // Weak, owned by our owner.
 #if defined(ENABLE_CONFIGURATION_POLICY)
   const policy::URLBlacklistManager* url_blacklist_manager_;
 #endif
   domain_reliability::DomainReliabilityMonitor* domain_reliability_monitor_;
 
   // When true, allow access to all file:// URLs.
   static bool g_allow_file_access_;
 
@@ skipping 26 matching lines @@
       data_reduction_proxy_usage_stats_;
   data_reduction_proxy::DataReductionProxyAuthRequestHandler*
   data_reduction_proxy_auth_request_handler_;
 
   OnResolveProxyHandler on_resolve_proxy_handler_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeNetworkDelegate);
 };
 
 #endif  // CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_