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

Unified Diff: net/log/net_log_event_type_list.h

Issue 1105443003: Add net_log events for the Data Reduction Proxy config service client. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: bengr CR comments Created 5 years, 8 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 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.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/log/net_log_event_type_list.h
diff --git a/net/log/net_log_event_type_list.h b/net/log/net_log_event_type_list.h
index c2cc7f25714bbbe8809780ac7c27bd820190c9e1..60f42a32e066cbe033b836c74e250a68bb153b46 100644
--- a/net/log/net_log_event_type_list.h
+++ b/net/log/net_log_event_type_list.h
@@ -2500,6 +2500,9 @@ EVENT_TYPE(DATA_REDUCTION_PROXY_ENABLED)
// The END phase contains the following parameters:
// {
// "net_error": <The net_error of the completion of the canary request>,
+// "http_response_code": <The HTTP response code of the canary request>,
+// "check_succeeded": <Whether a secure Data Reduction Proxy can be used or
+// not>
// }
EVENT_TYPE(DATA_REDUCTION_PROXY_CANARY_REQUEST)
@@ -2530,3 +2533,21 @@ EVENT_TYPE(DATA_REDUCTION_PROXY_BYPASS_REQUESTED)
// can be 0 if the proxy server is explicitly skipped>,
// }
EVENT_TYPE(DATA_REDUCTION_PROXY_FALLBACK)
+
+// The start/end of a config request is sent to the Data Saver Config API
+// service.
+//
+// The BEGIN phase contains the following parameters:
+// {
+// "url": <The URL of the service endpoint>,
+// }
+//
+// The END phase contains the following parameters:
+// {
+// "net_error": <The net_error of the completion of the config request>,
+// "http_response_code": <The HTTP response code of the config request>,
+// "failure_count": <The number of consecutive config request failures>,
+// "retry_delay_seconds": <The length of time after which another config
+// request will be made>,
+// }
+EVENT_TYPE(DATA_REDUCTION_PROXY_CONFIG_REQUEST)
« no previous file with comments | « components/data_reduction_proxy/core/common/data_reduction_proxy_event_store.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698