Return chrome-proxy-ect header code (post M59 branch)

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}
Committed: https://chromium.googlesource.com/chromium/src/+/211430bf1dc94f392d736cd02a3eed95b9f6dfaa

[dougarnett, 2017-04-17 19:59:08]

The CQ bit was checked by dougarnett@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-17 19:59:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-17 20:08:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-17 20:08:21]

Dry run: Try jobs failed on following builders:
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[dougarnett, 2017-04-17 21:14:18]

The CQ bit was checked by dougarnett@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-17 21:14:50]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-17 21:35:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-17 21:35:42]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[dougarnett, 2017-04-18 18:29:29]

The CQ bit was checked by dougarnett@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-18 18:30:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-18 19:19:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-18 19:19:45]

Dry run: This issue passed the CQ dry run.

[dougarnett, 2017-04-18 20:29:11]

dougarnett@chromium.org changed reviewers:
+ ryansturm@chromium.org

[dougarnett, 2017-04-18 20:29:12]

This puts back Tarun's ect header code plus a bit of integration with couple
more recently added unittests.

[RyanSturm, 2017-04-19 20:49:18]

lgtm % 1 test comment.

https://codereview.chromium.org/2825543002/diff/40001/components/data_reducti...
File
components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc
(right):

https://codereview.chromium.org/2825543002/diff/40001/components/data_reducti...
components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc:1520:
network_delegate()->NotifyBeforeSendHeaders(
Call NotifyBeforeStartTransaction again?

[dougarnett, 2017-04-19 21:19:19]

The CQ bit was checked by dougarnett@chromium.org to run a CQ dry run

[dougarnett, 2017-04-19 21:20:01]

https://codereview.chromium.org/2825543002/diff/40001/components/data_reducti...
File
components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc
(right):

https://codereview.chromium.org/2825543002/diff/40001/components/data_reducti...
components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc:1520:
network_delegate()->NotifyBeforeSendHeaders(
On 2017/04/19 20:49:18, Ryan Sturm wrote:
> Call NotifyBeforeStartTransaction again?

Done.

[commit-bot: I haz the power, 2017-04-19 21:20:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-19 22:16:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-19 22:16:09]

Dry run: This issue passed the CQ dry run.

[dougarnett, 2017-04-19 22:20:46]

The CQ bit was checked by dougarnett@chromium.org

[dougarnett, 2017-04-19 22:20:46]

The patchset sent to the CQ was uploaded after l-g-t-m from
ryansturm@chromium.org
Link to the patchset: https://codereview.chromium.org/2825543002/#ps60001
(title: "Update for comment by Ryan")

[commit-bot: I haz the power, 2017-04-19 22:21:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-19 22:26:49]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1492640446086630,
"parent_rev": "adebcac2524f6ec77c87bec594154961130822ed", "commit_rev":
"211430bf1dc94f392d736cd02a3eed95b9f6dfaa"}

[commit-bot: I haz the power, 2017-04-19 22:27:03]

Description was changed from

==========
Return chrome-proxy-ect header code (post M59 branch)

BUG=707981
==========

to

==========
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}
Committed:
https://chromium.googlesource.com/chromium/src/+/211430bf1dc94f392d736cd02a3e...
==========

[commit-bot: I haz the power, 2017-04-19 22:27:04]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/211430bf1dc94f392d736cd02a3e...