Data reduction proxy should record the time from initialization to the first data saver request. It will help in determining if it is useful for data reduction proxy to fetch a warm up URL.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3 commit e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3 Author: tbansal <tbansal@chromium.org> Date: Tue Nov 29 04:56:20 2016 Record time to first data reduction proxy request Record the time duration between data saver initialization and the first request whose resolved porxy is a data saver proxy. Also, record the time duration between IP address change event, and the next subsequent request whose resolved proxy is a data saver proxy. CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester BUG= 666134 Review-Url: https://codereview.chromium.org/2504963004 Cr-Commit-Position: refs/heads/master@{#434898} [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol_unittest.cc [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_service_client_unittest.cc [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.cc [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.h [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate_unittest.cc [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/components/data_reduction_proxy/core/browser/data_reduction_proxy_interceptor_unittest.cc [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data.cc [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data.h [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/components/data_reduction_proxy/core/browser/data_reduction_proxy_test_utils.cc [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/components/data_reduction_proxy/core/browser/data_reduction_proxy_test_utils.h [modify] https://crrev.com/e7e2d51baadbf0febbe84bfb8e50fe2840d3fbf3/tools/metrics/histograms/histograms.xml
Issue 589284 has been merged into this issue.
Comment 1 by tbansal@chromium.org
, Nov 17 2016