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

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 1702253002: Log more data reduction proxy config service UMA (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased, use boolean histogram Created 4 years, 10 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:
Download patch
« no previous file with comments | « components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 5f2f65bc9f4c309b10c54c899bc26158933387c5..8e0a16d16361217146bba45ea8bef2daa3574ef8 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -6564,6 +6564,18 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="DataReductionProxy.ConfigService.AuthFailure.LatencyPenalty"
+ units="ms">
+ <owner>bengr@chromium.org</owner>
+ <owner>tbansal@chromium.org</owner>
+ <summary>
+ Time from when the request starts to the time when the auth failure is
+ received from the data saver proxy.
+
+ Recorded every time an auth failure is received from the data saver proxy.
+ </summary>
+</histogram>
+
<histogram
name="DataReductionProxy.ConfigService.FetchFailedAttemptsBeforeSuccess">
<owner>bengr@chromium.org</owner>
@@ -6596,6 +6608,21 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="DataReductionProxy.ConfigService.HTTPRequests"
+ enum="DataReductionProxyConfigServiceHTTPRequests">
+ <owner>bengr@chromium.org</owner>
+ <owner>tbansal@chromium.org</owner>
+ <summary>
+ Upper bound on the count of HTTP requests that did or did not go through the
+ data saver proxy.
+
+ An HTTP request may not go through data saver proxy if the config was
+ unavailable, local bypass rules or server side bypasses. If data saver is
+ turned on, this metric is recorded every time proxy is resolved for an HTTP
+ request.
+ </summary>
+</histogram>
+
<histogram name="DataReductionProxy.EnabledState"
enum="DataReductionProxyEnabledState">
<owner>bengr@chromium.org</owner>
@@ -60865,6 +60892,11 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<int value="1" label="Current session key matches the key in the request"/>
</enum>
+<enum name="DataReductionProxyConfigServiceHTTPRequests" type="int">
+ <int value="0" label="Request went through data saver proxy"/>
+ <int value="1" label="Request did not go through data saver proxy"/>
+</enum>
+
<enum name="DataReductionProxyEnabledState" type="int">
<int value="0" label="Off-to-On"/>
<int value="1" label="On-to-Off"/>
@@ -83460,7 +83492,8 @@ To add a new entry, add it with any value and run test to compute valid value.
<histogram_suffixes name="DataReductionProxyBypassedBytes" separator=".">
<suffix name="SSL" label="Bypass due to SSL"/>
<suffix name="LocalBypassRules"
- label="Bypass due to client-side bypass rules"/>
+ label="Bypass due to client-side bypass rules or because the proxy
+ config was missing at the client"/>
<suffix name="ManagedProxyConfig"
label="Deprecated 1/9/2014 (M-41). Bypass due to a managed config"/>
<suffix name="ProxyOverridden"
« no previous file with comments | « components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698