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

Unified Diff: components/data_reduction_proxy/core/common/data_reduction_proxy_headers.h

Issue 775773002: Add data reduction proxy debug info to net-internals#bandwidth (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: bengr CR updates Created 6 years 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/data_reduction_proxy/core/common/data_reduction_proxy_event_store_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/data_reduction_proxy/core/common/data_reduction_proxy_headers.h
diff --git a/components/data_reduction_proxy/core/common/data_reduction_proxy_headers.h b/components/data_reduction_proxy/core/common/data_reduction_proxy_headers.h
index b4d8ec8c1a37dcd8777e1ef7cf097ebfeaf24552..3df14963520c6bbbb09366586ca51d5f9c3f407f 100644
--- a/components/data_reduction_proxy/core/common/data_reduction_proxy_headers.h
+++ b/components/data_reduction_proxy/core/common/data_reduction_proxy_headers.h
@@ -23,46 +23,14 @@ namespace data_reduction_proxy {
class DataReductionProxyEventStore;
-// Values of the UMA DataReductionProxy.BypassType{Primary|Fallback}
-// and DataReductionProxy.BlockType{Primary|Fallback} histograms.
-// This enum must remain synchronized with the enum of the same
-// name in metrics/histograms/histograms.xml.
+// Values of the UMA DataReductionProxy.BypassType{Primary|Fallback} and
+// DataReductionProxy.BlockType{Primary|Fallback} histograms. This enum must
+// remain synchronized with the enum of the same name in
+// metrics/histograms/histograms.xml.
enum DataReductionProxyBypassType {
- // Bypass due to explicit instruction for the current request.
- BYPASS_EVENT_TYPE_CURRENT = 0,
-
- // Bypass the proxy for less than one minute.
- BYPASS_EVENT_TYPE_SHORT = 1,
-
- // Bypass the proxy for one to five minutes.
- BYPASS_EVENT_TYPE_MEDIUM = 2,
-
- // Bypass the proxy for more than five minutes.
- BYPASS_EVENT_TYPE_LONG = 3,
-
- // Bypass due to a 4xx missing via header.
- BYPASS_EVENT_TYPE_MISSING_VIA_HEADER_4XX = 4,
-
- // Bypass due to other missing via header, excluding 4xx errors.
- BYPASS_EVENT_TYPE_MISSING_VIA_HEADER_OTHER = 5,
-
- // Bypass due to 407 response from proxy without a challenge.
- BYPASS_EVENT_TYPE_MALFORMED_407 = 6,
-
- // Bypass due to a 500 internal server error
- BYPASS_EVENT_TYPE_STATUS_500_HTTP_INTERNAL_SERVER_ERROR = 7,
-
- // Bypass because the request URI was too long.
- BYPASS_EVENT_TYPE_STATUS_502_HTTP_BAD_GATEWAY = 8,
-
- // Bypass due to a 503 response.
- BYPASS_EVENT_TYPE_STATUS_503_HTTP_SERVICE_UNAVAILABLE = 9,
-
- // Bypass due to any network error.
- BYPASS_EVENT_TYPE_NETWORK_ERROR = 10,
-
- // This must always be last.
- BYPASS_EVENT_TYPE_MAX = 11
+#define BYPASS_EVENT_TYPE(label, value) BYPASS_EVENT_TYPE_ ## label = value,
+#include "components/data_reduction_proxy/core/common/data_reduction_proxy_bypass_type_list.h"
+#undef BYPASS_EVENT_TYPE
};
// Contains instructions contained in the Chrome-Proxy header.
« no previous file with comments | « components/data_reduction_proxy/core/common/data_reduction_proxy_event_store_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698