Bypassed Bytes UMAs

components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.h

Index: components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.h
diff --git a/components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.h b/components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.h
index 6c217b2f5768bdba67c736494cee83e2629dc718..4e3dbfe6fb891ae0120428180c030a49cb0db12a 100644
--- a/components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.h
+++ b/components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.h
@@ -6,10 +6,13 @@
 #define COMPONENTS_DATA_REDUCTION_PROXY_BROWSER_DATA_REDUCTION_PROXY_USAGE_STATS_H_
 
 #include "base/message_loop/message_loop_proxy.h"
+#include "base/prefs/pref_member.h"
 #include "base/threading/thread_checker.h"
 #include "components/data_reduction_proxy/browser/data_reduction_proxy_params.h"
 #include "net/base/host_port_pair.h"
 #include "net/base/network_change_notifier.h"
+#include "net/proxy/proxy_service.h"
+#include "net/url_request/url_request.h"
 
 namespace data_reduction_proxy {
 
@@ -23,20 +26,44 @@ class DataReductionProxyUsageStats
                                base::MessageLoopProxy* io_thread_proxy);
   virtual ~DataReductionProxyUsageStats();
 
-  /**
-   * Callback intended to be called from |ChromeNetworkDelegate| when a
-   * request completes. This method is used to gather usage stats.
-   */
+  // Callback intended to be called from |ChromeNetworkDelegate| when a
+  // request completes. This method is used to gather usage stats.
   void OnUrlRequestCompleted(const net::URLRequest* request, bool started);
 
-  /**
-   * Determines whether the data reduction proxy is unreachable.
-   * Returns true if data reduction proxy is unreachable.
-   */
+  // Determines whether the data reduction proxy is unreachable.
+  // Returns true if data reduction proxy is unreachable.
   bool isDataReductionProxyUnreachable();
 
+  // Records the last bypass reason to |bypass_type_| and sets
+  // |triggering_request_| to true. A triggering request is the first request to
+  // cause the current bypass.
+  void SetBypassType(net::ProxyService::DataReductionProxyBypassType type);
+
+  // Given the |content_length| and associated |request|, records the
+  // number of bypassed bytes for that |request| into UMAs based on bypass type.
+  // |data_reduction_proxy_enabled| tells us the state of the
+  // kDataReductionProxyEnabled preference.
+  void RecordBypassedBytesHistograms(
+      net::URLRequest& request,
+      const BooleanPrefMember& data_reduction_proxy_enabled);
+
  private:
+  enum BypassedBytesType {
+    NOT_BYPASSED = 0,         /* Not bypassed. */
+    SSL,                      /* Bypass due to SSL. */
+    LOCAL_BYPASS_RULES,       /* Bypass due to client-side bypass rules. */
+    AUDIO_VIDEO,              /* Audio/Video bypass. */
+    TRIGGERING_REQUEST,       /* Triggering request bypass. */
+    NETWORK_ERROR,            /* Network error. */
+    BYPASSED_BYTES_TYPE_MAX   /* This must always be last.*/
+  };
+
   DataReductionProxyParams* data_reduction_proxy_params_;
+  // The last reason for bypass as determined by
+  // MaybeBypassProxyAndPrepareToRetry
+  net::ProxyService::DataReductionProxyBypassType last_bypass_type_;
+  // True if the last request triggered the current bypass.
+  bool triggering_request_;
   base::MessageLoopProxy* ui_thread_proxy_;
   base::MessageLoopProxy* io_thread_proxy_;
 
@@ -63,6 +90,11 @@ class DataReductionProxyUsageStats
 
   base::ThreadChecker thread_checker_;
 
+  void RecordBypassedBytes(
+      net::ProxyService::DataReductionProxyBypassType bypass_type,
+      BypassedBytesType bypassed_bytes_type,
+      int64 content_length);
+
   DISALLOW_COPY_AND_ASSIGN(DataReductionProxyUsageStats);
 };