Whether or not to add the Chrome-Proxy-ECT header should be behind a flag, with default off in M-59.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/46aa9d9db9c0581937a96ced13747039fffe943c commit 46aa9d9db9c0581937a96ced13747039fffe943c Author: dougarnett <dougarnett@chromium.org> Date: Thu Apr 06 21:45:19 2017 Removes the chrome-proxy-ect header from M59. This reverts sending the ECT header to the proxy server. Note: doing this with fresh CL seems cleaner that backing out the original 3 CLs - especially because we intend to revert this CL to put back the ECT header after the M59 branch is cut. [This is an alternative to gating behind a Feature in cl 2802463002] BUG= 707981 Review-Url: https://codereview.chromium.org/2807453002 Cr-Commit-Position: refs/heads/master@{#462632} [modify] https://crrev.com/46aa9d9db9c0581937a96ced13747039fffe943c/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.cc [modify] https://crrev.com/46aa9d9db9c0581937a96ced13747039fffe943c/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.h [modify] https://crrev.com/46aa9d9db9c0581937a96ced13747039fffe943c/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc [modify] https://crrev.com/46aa9d9db9c0581937a96ced13747039fffe943c/components/data_reduction_proxy/core/common/data_reduction_proxy_headers.cc [modify] https://crrev.com/46aa9d9db9c0581937a96ced13747039fffe943c/components/data_reduction_proxy/core/common/data_reduction_proxy_headers.h
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/211430bf1dc94f392d736cd02a3eed95b9f6dfaa commit 211430bf1dc94f392d736cd02a3eed95b9f6dfaa Author: dougarnett <dougarnett@chromium.org> Date: Wed Apr 19 22:26:45 2017 Return chrome-proxy-ect header code (post M59 branch) BUG= 707981 Review-Url: https://codereview.chromium.org/2825543002 Cr-Commit-Position: refs/heads/master@{#465783} [modify] https://crrev.com/211430bf1dc94f392d736cd02a3eed95b9f6dfaa/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.cc [modify] https://crrev.com/211430bf1dc94f392d736cd02a3eed95b9f6dfaa/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.h [modify] https://crrev.com/211430bf1dc94f392d736cd02a3eed95b9f6dfaa/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc [modify] https://crrev.com/211430bf1dc94f392d736cd02a3eed95b9f6dfaa/components/data_reduction_proxy/core/common/data_reduction_proxy_headers.cc [modify] https://crrev.com/211430bf1dc94f392d736cd02a3eed95b9f6dfaa/components/data_reduction_proxy/core/common/data_reduction_proxy_headers.h
Comment 1 by bugdroid1@chromium.org
, Apr 6 2017