Change MISSING_VIA_HEADER_4XX bypasses to only last 1 second.

Change MISSING_VIA_HEADER_4XX bypasses to only last 1 second.

Reduce the duration of MISSING_VIA_HEADER_4XX data reduction proxy bypasses down to 1 second instead of 1-5 minutes, since any 4xx error that could occur at that point would be scoped to just the current request.

BUG=422526
Committed: https://crrev.com/8c2273d6551e93e5d47806b0aeb9e1abcf4ee1d5
Cr-Commit-Position: refs/heads/master@{#299222}

[sclittle, 2014-10-10 21:46:57]

sclittle@chromium.org changed reviewers:
+ bengr@chromium.org

[sclittle, 2014-10-10 21:46:58]

bengr: all

Thanks in advance!

[bengr, 2014-10-10 21:52:50]

https://codereview.chromium.org/651433003/diff/1/components/data_reduction_pr...
File components/data_reduction_proxy/common/data_reduction_proxy_headers.cc
(right):

https://codereview.chromium.org/651433003/diff/1/components/data_reduction_pr...
components/data_reduction_proxy/common/data_reduction_proxy_headers.cc:248:
data_reduction_proxy_info->bypass_duration = TimeDelta::FromSeconds(1);
There should be a test for this somewhere, either in headers_unittest or
protocol_unittest. If this case isn't covered, please add it.

[sclittle, 2014-10-10 22:02:08]

https://codereview.chromium.org/651433003/diff/1/components/data_reduction_pr...
File components/data_reduction_proxy/common/data_reduction_proxy_headers.cc
(right):

https://codereview.chromium.org/651433003/diff/1/components/data_reduction_pr...
components/data_reduction_proxy/common/data_reduction_proxy_headers.cc:248:
data_reduction_proxy_info->bypass_duration = TimeDelta::FromSeconds(1);
On 2014/10/10 21:52:50, bengr1 wrote:
> There should be a test for this somewhere, either in headers_unittest or
> protocol_unittest. If this case isn't covered, please add it.

Done.

[bengr, 2014-10-10 23:20:05]

lgtm. Please specify in the CL description or title that this change has to do
with the data reduction proxy.

[sclittle, 2014-10-10 23:22:11]

On 2014/10/10 23:20:05, bengr1 wrote:
> lgtm. Please specify in the CL description or title that this change has to do
> with the data reduction proxy.

I've updated the CL description accordingly.

[sclittle, 2014-10-10 23:22:15]

The CQ bit was checked by sclittle@chromium.org

[commit-bot: I haz the power, 2014-10-10 23:23:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/651433003/70001

[commit-bot: I haz the power, 2014-10-11 00:52:18]

Committed patchset #2 (id:70001)

[commit-bot: I haz the power, 2014-10-11 00:52:53]

Patchset 2 (id:??) landed as
https://crrev.com/8c2273d6551e93e5d47806b0aeb9e1abcf4ee1d5
Cr-Commit-Position: refs/heads/master@{#299222}