Make warmup URL probe timeout configurable via finch |
||||
Issue descriptionCurrently, we use a timeout value of 5 times the HTTP RTT estimate. This value is pretty low. We should make the timeout value configurable via finch.
,
Apr 6 2018
,
Apr 7 2018
,
Apr 7 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cd3c776f5ebc7acff482a2349bcedb7b8330dc1d commit cd3c776f5ebc7acff482a2349bcedb7b8330dc1d Author: Tarun Bansal <tbansal@chromium.org> Date: Sat Apr 07 01:18:14 2018 Make warmup URL probe timeout configurable via finch Also, add histogram to record probe fetch attempt count There is no functional change introduced by this CL. Bug: 830088 Change-Id: I5aaca33dbec938b5a0462b3daf5b64dcdfeff7d1 Reviewed-on: https://chromium-review.googlesource.com/1000550 Commit-Queue: Tarun Bansal <tbansal@chromium.org> Reviewed-by: Ryan Sturm <ryansturm@chromium.org> Cr-Commit-Position: refs/heads/master@{#549004} [modify] https://crrev.com/cd3c776f5ebc7acff482a2349bcedb7b8330dc1d/components/data_reduction_proxy/core/browser/data_reduction_proxy_configurator_unittest.cc [modify] https://crrev.com/cd3c776f5ebc7acff482a2349bcedb7b8330dc1d/components/data_reduction_proxy/core/browser/network_properties_manager.cc [modify] https://crrev.com/cd3c776f5ebc7acff482a2349bcedb7b8330dc1d/components/data_reduction_proxy/core/browser/warmup_url_fetcher.cc [modify] https://crrev.com/cd3c776f5ebc7acff482a2349bcedb7b8330dc1d/components/data_reduction_proxy/core/browser/warmup_url_fetcher_unittest.cc
,
Apr 11 2018
,
Apr 20 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/af0b0900b5d6bf71f14337ad7bf666d5ae6469dc commit af0b0900b5d6bf71f14337ad7bf666d5ae6469dc Author: Tarun Bansal <tbansal@chromium.org> Date: Fri Apr 20 19:42:21 2018 Add histograms for data reduction proxy warmup URL to histograms.xml These histograms were added in a previous CL (http://shortn/_1VKvI480Ld), but I forgot to add them to histograms.xml. Bug: 830088 Change-Id: I8496ee47d11d0e631df46fff3706b3b668bba570 Reviewed-on: https://chromium-review.googlesource.com/1022230 Reviewed-by: Ryan Sturm <ryansturm@chromium.org> Reviewed-by: Jesse Doherty <jwd@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#552435} [modify] https://crrev.com/af0b0900b5d6bf71f14337ad7bf666d5ae6469dc/tools/metrics/histograms/histograms.xml |
||||
►
Sign in to add a comment |
||||
Comment 1 by tbansal@chromium.org
, Apr 6 2018Components: Internals>Network>DataProxy
Labels: OS-Android OS-Chrome OS-Linux OS-Mac OS-Windows