[wpr-go] Match NegotiatedProtocol

[wpr-go] Match NegotiatedProtocol

Use a tls.Config with NextProtos that match the protocol used by
the original server. If Chrome negotiated h2, the replay
server should also speak h2 when the resource is requested. If
h2 is not used in recording mode, replay mode should default to
http/1.1.

BUG=catapult:#3674

Review-Url: https://codereview.chromium.org/2956123002
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/b0300e32745103bd6cf8ed6c4e54e524c22c1392

[xunjieli, 2017-06-27 19:48:56]

Description was changed from

==========
[wpr-go] Match NegotiatedProtocol

BUG=catapult:#
==========

to

==========
[wpr-go] Match NegotiatedProtocol

BUG=catapult:#3674
==========

[xunjieli, 2017-06-27 19:59:51]

Description was changed from

==========
[wpr-go] Match NegotiatedProtocol

BUG=catapult:#3674
==========

to

==========
[wpr-go] Match NegotiatedProtocol

Use a tls.Config with NextProtos that match the protocol used by
the original server. If Chrome negotiated h2, the replay
server should also speak h2 when the resource is requested. If
h2 is not used in recording mode, replay mode should default to
http/1.1.

BUG=catapult:#3674
==========

[xunjieli, 2017-06-27 20:00:03]

Patchset #2 (id:20001) has been deleted

[xunjieli, 2017-06-27 20:00:08]

Patchset #2 (id:40001) has been deleted

[xunjieli, 2017-06-27 20:00:14]

Patchset #1 (id:1) has been deleted

[xunjieli, 2017-06-27 20:00:45]

xunjieli@chromium.org changed reviewers:
+ nednguyen@google.com, tombergan@chromium.org

[xunjieli, 2017-06-27 20:00:46]

PTAL. Thank you.

[Tom Bergan, 2017-06-27 21:37:02]

lgtm

[xunjieli, 2017-06-28 13:31:15]

I will check in this now. Ned told me he is busy with OKR these few days. 
We don't have test coverage for this part of the code yet, but I will work on
it.

[xunjieli, 2017-06-28 13:31:20]

The CQ bit was checked by xunjieli@chromium.org

[commit-bot: I haz the power, 2017-06-28 13:31:27]

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-06-28 14:32:55]

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

[commit-bot: I haz the power, 2017-06-28 14:32:56]

Try jobs failed on following builders:
  Catapult Linux Tryserver on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Li...)

[xunjieli, 2017-06-28 14:43:12]

The CQ bit was checked by xunjieli@chromium.org

[commit-bot: I haz the power, 2017-06-28 14:43:17]

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-06-28 14:57:37]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1498660992469360,
"parent_rev": "a1cf261018f3e8ef1ec2beb9b2a89b12b8c4c0c7", "commit_rev":
"b0300e32745103bd6cf8ed6c4e54e524c22c1392"}

[commit-bot: I haz the power, 2017-06-28 14:57:40]

Description was changed from

==========
[wpr-go] Match NegotiatedProtocol

Use a tls.Config with NextProtos that match the protocol used by
the original server. If Chrome negotiated h2, the replay
server should also speak h2 when the resource is requested. If
h2 is not used in recording mode, replay mode should default to
http/1.1.

BUG=catapult:#3674
==========

to

==========
[wpr-go] Match NegotiatedProtocol

Use a tls.Config with NextProtos that match the protocol used by
the original server. If Chrome negotiated h2, the replay
server should also speak h2 when the resource is requested. If
h2 is not used in recording mode, replay mode should default to
http/1.1.

BUG=catapult:#3674

Review-Url: https://codereview.chromium.org/2956123002
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-06-28 14:57:41]

Committed patchset #1 (id:60001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...