Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(160)

Side by Side Diff: components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.h

Issue 390533003: Bypassed Bytes UMAs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Moving #if defined(SPDY_PROXY_AUTH_ORIGIN) Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_DATA_REDUCTION_PROXY_BROWSER_DATA_REDUCTION_PROXY_USAGE_STATS _H_ 5 #ifndef COMPONENTS_DATA_REDUCTION_PROXY_BROWSER_DATA_REDUCTION_PROXY_USAGE_STATS _H_
6 #define COMPONENTS_DATA_REDUCTION_PROXY_BROWSER_DATA_REDUCTION_PROXY_USAGE_STATS _H_ 6 #define COMPONENTS_DATA_REDUCTION_PROXY_BROWSER_DATA_REDUCTION_PROXY_USAGE_STATS _H_
7 7
8 #include "base/message_loop/message_loop_proxy.h" 8 #include "base/message_loop/message_loop_proxy.h"
9 #include "base/prefs/pref_member.h"
9 #include "base/threading/thread_checker.h" 10 #include "base/threading/thread_checker.h"
10 #include "components/data_reduction_proxy/browser/data_reduction_proxy_params.h" 11 #include "components/data_reduction_proxy/browser/data_reduction_proxy_params.h"
11 #include "net/base/host_port_pair.h" 12 #include "net/base/host_port_pair.h"
12 #include "net/base/network_change_notifier.h" 13 #include "net/base/network_change_notifier.h"
14 #include "net/proxy/proxy_service.h"
15 #include "net/url_request/url_request.h"
13 16
14 namespace data_reduction_proxy { 17 namespace data_reduction_proxy {
15 18
16 class DataReductionProxyUsageStats 19 class DataReductionProxyUsageStats
17 : public net::NetworkChangeNotifier::NetworkChangeObserver { 20 : public net::NetworkChangeNotifier::NetworkChangeObserver {
18 public: 21 public:
19 // MessageLoopProxy instances are owned by io_thread. |params| outlives 22 // MessageLoopProxy instances are owned by io_thread. |params| outlives
20 // this class instance. 23 // this class instance.
21 DataReductionProxyUsageStats(DataReductionProxyParams* params, 24 DataReductionProxyUsageStats(DataReductionProxyParams* params,
22 base::MessageLoopProxy* ui_thread_proxy, 25 base::MessageLoopProxy* ui_thread_proxy,
23 base::MessageLoopProxy* io_thread_proxy); 26 base::MessageLoopProxy* io_thread_proxy);
24 virtual ~DataReductionProxyUsageStats(); 27 virtual ~DataReductionProxyUsageStats();
25 28
26 /** 29 // Callback intended to be called from |ChromeNetworkDelegate| when a
27 * Callback intended to be called from |ChromeNetworkDelegate| when a 30 // request completes. This method is used to gather usage stats.
28 * request completes. This method is used to gather usage stats.
29 */
30 void OnUrlRequestCompleted(const net::URLRequest* request, bool started); 31 void OnUrlRequestCompleted(const net::URLRequest* request, bool started);
31 32
32 /** 33 // Determines whether the data reduction proxy is unreachable.
33 * Determines whether the data reduction proxy is unreachable. 34 // Returns true if data reduction proxy is unreachable.
34 * Returns true if data reduction proxy is unreachable.
35 */
36 bool isDataReductionProxyUnreachable(); 35 bool isDataReductionProxyUnreachable();
37 36
37 // Records the last bypass reason to |bypass_type_| and sets
38 // |triggering_request_| to true. A triggering request is the first request to
39 // cause the current bypass.
40 void SetBypassType(net::ProxyService::DataReductionProxyBypassType type);
41
42 // Given the |content_length| and associated |request|, records the
43 // number of bypassed bytes for that |request| into UMAs based on bypass type.
44 // |data_reduction_proxy_enabled| tells us the state of the
45 // kDataReductionProxyEnabled preference.
46 void RecordBypassedBytesHistograms(
47 net::URLRequest& request,
48 const BooleanPrefMember& data_reduction_proxy_enabled);
49
38 private: 50 private:
51 enum BypassedBytesType {
52 NOT_BYPASSED = 0, /* Not bypassed. */
53 SSL, /* Bypass due to SSL. */
54 LOCAL_BYPASS_RULES, /* Bypass due to client-side bypass rules. */
55 AUDIO_VIDEO, /* Audio/Video bypass. */
56 TRIGGERING_REQUEST, /* Triggering request bypass. */
57 NETWORK_ERROR, /* Network error. */
58 BYPASSED_BYTES_TYPE_MAX /* This must always be last.*/
59 };
60
39 DataReductionProxyParams* data_reduction_proxy_params_; 61 DataReductionProxyParams* data_reduction_proxy_params_;
62 // The last reason for bypass as determined by
63 // MaybeBypassProxyAndPrepareToRetry
64 net::ProxyService::DataReductionProxyBypassType last_bypass_type_;
65 // True if the last request triggered the current bypass.
66 bool triggering_request_;
40 base::MessageLoopProxy* ui_thread_proxy_; 67 base::MessageLoopProxy* ui_thread_proxy_;
41 base::MessageLoopProxy* io_thread_proxy_; 68 base::MessageLoopProxy* io_thread_proxy_;
42 69
43 // The following 2 fields are used to determine if data reduction proxy is 70 // The following 2 fields are used to determine if data reduction proxy is
44 // unreachable. We keep a count of requests which should go through 71 // unreachable. We keep a count of requests which should go through
45 // data request proxy, as well as those which actually do. The proxy is 72 // data request proxy, as well as those which actually do. The proxy is
46 // unreachable if no successful requests are made through it despite a 73 // unreachable if no successful requests are made through it despite a
47 // non-zero number of requests being eligible. 74 // non-zero number of requests being eligible.
48 75
49 // Count of requests which will be tried to be sent through data reduction 76 // Count of requests which will be tried to be sent through data reduction
50 // proxy. The count is only based on the config and not the bad proxy list. 77 // proxy. The count is only based on the config and not the bad proxy list.
51 // Explicit bypasses are not part of this count. This is the desired behavior 78 // Explicit bypasses are not part of this count. This is the desired behavior
52 // since otherwise both counts would be identical. 79 // since otherwise both counts would be identical.
53 unsigned long eligible_num_requests_through_proxy_; 80 unsigned long eligible_num_requests_through_proxy_;
54 // Count of successfull requests through data reduction proxy. 81 // Count of successfull requests through data reduction proxy.
55 unsigned long actual_num_requests_through_proxy_; 82 unsigned long actual_num_requests_through_proxy_;
56 83
57 // NetworkChangeNotifier::NetworkChangeObserver: 84 // NetworkChangeNotifier::NetworkChangeObserver:
58 virtual void OnNetworkChanged( 85 virtual void OnNetworkChanged(
59 net::NetworkChangeNotifier::ConnectionType type) OVERRIDE; 86 net::NetworkChangeNotifier::ConnectionType type) OVERRIDE;
60 87
61 void IncRequestCountsOnUiThread(bool actual); 88 void IncRequestCountsOnUiThread(bool actual);
62 void ClearRequestCountsOnUiThread(); 89 void ClearRequestCountsOnUiThread();
63 90
64 base::ThreadChecker thread_checker_; 91 base::ThreadChecker thread_checker_;
65 92
93 void RecordBypassedBytes(
94 net::ProxyService::DataReductionProxyBypassType bypass_type,
95 BypassedBytesType bypassed_bytes_type,
96 int64 content_length);
97
66 DISALLOW_COPY_AND_ASSIGN(DataReductionProxyUsageStats); 98 DISALLOW_COPY_AND_ASSIGN(DataReductionProxyUsageStats);
67 }; 99 };
68 100
69 } // namespace data_reduction_proxy 101 } // namespace data_reduction_proxy
70 102
71 #endif // COMPONENTS_DATA_REDUCTION_PROXY_BROWSER_DATA_REDUCTION_PROXY_USAGE_ST ATS_H_ 103 #endif // COMPONENTS_DATA_REDUCTION_PROXY_BROWSER_DATA_REDUCTION_PROXY_USAGE_ST ATS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698