Landing Recent QUIC changes until 5:30 PM, Feb 3, 2017 UTC-5

Landing Recent QUIC changes until 5:30 PM, Feb 3, 2017 UTC-5

Flip FLAGS_quic_reloadable_flag_quic_use_event_time to true.
https://codereview.chromium.org/2680803002/

Remove path id from IV in QUIC encrytion algorithms.
Merge internal change: 146517512
https://codereview.chromium.org/2679673005/

Get rid of more va_arg methods in favor of std::vector<> arguments. Much simpler and more readable method implementations as a result.
Merge internal change: 146383132
https://codereview.chromium.org/2671273003/

Remove two unused ProofSourceForTesting.
Merge internal change: 146291369
https://codereview.chromium.org/2676343004/

Replace va_arg version of crypto_test_utils::Message with a more modern vector argument version.
Couple of other simplifications:
- always build a CHLO (this method was never used for anything else)
- make padding an explicit argument rather than a weird special case
- remove disabling of clang-format as things look good with this change
Merge internal change: 146270645
https://codereview.chromium.org/2678973002/

Deprecate FLAGS_quic_reloadable_flag_enable_async_get_proof
Merge internal change: 146243587
https://codereview.chromium.org/2679783003/

Deprecate FLAGS_quic_reloadable_flag_quic_only_5rto_client_side in the disabled state, now that it's been disabled for over a month.
Merge internal change: 146241134
https://codereview.chromium.org/2678953002/

Convert class-full-of-statics CryptoTestUtils into a namespace
Merge internal change: 146240722
https://codereview.chromium.org/2671333004/

Deprecate FLAGS_quic_reloadable_flag_quic_more_conservative_retransmission_alarm
Merge internal change: 146149718
https://codereview.chromium.org/2671303003/

Replace crypto::SecureHash with calls to SHA256.
Merge internal change: 146047815
https://codereview.chromium.org/2672403003/

Make QUIC force HOL-blocking mode only apply to v36. This is basically a no-op since neither v36 nor v37 are enabled.
Merge internal change: 146004954
https://codereview.chromium.org/2678003003/

Review-Url: https://codereview.chromium.org/2681793002
Cr-Commit-Position: refs/heads/master@{#448645}
Committed: https://chromium.googlesource.com/chromium/src/+/f5f4cd51c92ce55d2eaadd371c6aa1ad0aa2812e

[Fan Yang, 2017-02-07 14:58:08]

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

[commit-bot: I haz the power, 2017-02-07 14:58:23]

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

[Fan Yang, 2017-02-07 15:16:58]

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

[Fan Yang, 2017-02-07 15:17:10]

Description was changed from

==========
Landing Recent QUIC changes until 5:30 PM, Feb 3, 2017 UTC-5

Flip FLAGS_quic_reloadable_flag_quic_use_event_time to true.
https://codereview.chromium.org/2680803002/

Remove path id from IV in QUIC encrytion algorithms.
Merge internal change: 146517512
https://codereview.chromium.org/2679673005/

Get rid of more va_arg methods in favor of std::vector<> arguments. Much simpler
and more readable method implementations as a result.
Merge internal change: 146383132
https://codereview.chromium.org/2671273003/

Remove two unused ProofSourceForTesting.
Merge internal change: 146291369
https://codereview.chromium.org/2676343004/

Replace va_arg version of crypto_test_utils::Message with a more modern vector
argument version.
Couple of other simplifications:
- always build a CHLO (this method was never used for anything else)
- make padding an explicit argument rather than a weird special case
- remove disabling of clang-format as things look good with this change
Merge internal change: 146270645
https://codereview.chromium.org/2678973002/

Deprecate FLAGS_quic_reloadable_flag_enable_async_get_proof
Merge internal change: 146243587
https://codereview.chromium.org/2679783003/

Deprecate FLAGS_quic_reloadable_flag_quic_only_5rto_client_side in the disabled
state, now that it's been disabled for over a month.
Merge internal change: 146241134
https://codereview.chromium.org/2678953002/

Convert class-full-of-statics CryptoTestUtils into a namespace
Merge internal change: 146240722
https://codereview.chromium.org/2671333004/

Deprecate FLAGS_quic_reloadable_flag_quic_more_conservative_retransmission_alarm
Merge internal change: 146149718
https://codereview.chromium.org/2671303003/

Replace crypto::SecureHash with calls to SHA256.
Merge internal change: 146047815
https://codereview.chromium.org/2672403003/

Make QUIC force HOL-blocking mode only apply to v36. This is basically a no-op
since neither v36 nor v37 are enabled.
Merge internal change: 146004954
https://codereview.chromium.org/2678003003/
==========

to

==========
Landing Recent QUIC changes until 5:30 PM, Feb 3, 2017 UTC-5

Flip FLAGS_quic_reloadable_flag_quic_use_event_time to true.
https://codereview.chromium.org/2680803002/

Remove path id from IV in QUIC encrytion algorithms.
Merge internal change: 146517512
https://codereview.chromium.org/2679673005/

Get rid of more va_arg methods in favor of std::vector<> arguments. Much simpler
and more readable method implementations as a result.
Merge internal change: 146383132
https://codereview.chromium.org/2671273003/

Remove two unused ProofSourceForTesting.
Merge internal change: 146291369
https://codereview.chromium.org/2676343004/

Replace va_arg version of crypto_test_utils::Message with a more modern vector
argument version.
Couple of other simplifications:
- always build a CHLO (this method was never used for anything else)
- make padding an explicit argument rather than a weird special case
- remove disabling of clang-format as things look good with this change
Merge internal change: 146270645
https://codereview.chromium.org/2678973002/

Deprecate FLAGS_quic_reloadable_flag_enable_async_get_proof
Merge internal change: 146243587
https://codereview.chromium.org/2679783003/

Deprecate FLAGS_quic_reloadable_flag_quic_only_5rto_client_side in the disabled
state, now that it's been disabled for over a month.
Merge internal change: 146241134
https://codereview.chromium.org/2678953002/

Convert class-full-of-statics CryptoTestUtils into a namespace
Merge internal change: 146240722
https://codereview.chromium.org/2671333004/

Deprecate FLAGS_quic_reloadable_flag_quic_more_conservative_retransmission_alarm
Merge internal change: 146149718
https://codereview.chromium.org/2671303003/

Replace crypto::SecureHash with calls to SHA256.
Merge internal change: 146047815
https://codereview.chromium.org/2672403003/

Make QUIC force HOL-blocking mode only apply to v36. This is basically a no-op
since neither v36 nor v37 are enabled.
Merge internal change: 146004954
https://codereview.chromium.org/2678003003/
==========

[Fan Yang, 2017-02-07 15:17:10]

fayang@chromium.org changed reviewers:
+ rch@chromium.org

[commit-bot: I haz the power, 2017-02-07 15:17:16]

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

[Ryan Hamilton, 2017-02-07 15:24:44]

lgtm

[commit-bot: I haz the power, 2017-02-07 16:35:27]

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

[commit-bot: I haz the power, 2017-02-07 16:35:28]

Dry run: This issue passed the CQ dry run.

[Fan Yang, 2017-02-07 16:50:42]

The CQ bit was checked by fayang@chromium.org

[commit-bot: I haz the power, 2017-02-07 16:51:03]

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-02-07 16:57:17]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486486242363360,
"parent_rev": "3dae9e7b5e6ab5604b8f32916ab0a45a59eeb269", "commit_rev":
"f5f4cd51c92ce55d2eaadd371c6aa1ad0aa2812e"}

[commit-bot: I haz the power, 2017-02-07 16:57:48]

Description was changed from

==========
Landing Recent QUIC changes until 5:30 PM, Feb 3, 2017 UTC-5

Flip FLAGS_quic_reloadable_flag_quic_use_event_time to true.
https://codereview.chromium.org/2680803002/

Remove path id from IV in QUIC encrytion algorithms.
Merge internal change: 146517512
https://codereview.chromium.org/2679673005/

Get rid of more va_arg methods in favor of std::vector<> arguments. Much simpler
and more readable method implementations as a result.
Merge internal change: 146383132
https://codereview.chromium.org/2671273003/

Remove two unused ProofSourceForTesting.
Merge internal change: 146291369
https://codereview.chromium.org/2676343004/

Replace va_arg version of crypto_test_utils::Message with a more modern vector
argument version.
Couple of other simplifications:
- always build a CHLO (this method was never used for anything else)
- make padding an explicit argument rather than a weird special case
- remove disabling of clang-format as things look good with this change
Merge internal change: 146270645
https://codereview.chromium.org/2678973002/

Deprecate FLAGS_quic_reloadable_flag_enable_async_get_proof
Merge internal change: 146243587
https://codereview.chromium.org/2679783003/

Deprecate FLAGS_quic_reloadable_flag_quic_only_5rto_client_side in the disabled
state, now that it's been disabled for over a month.
Merge internal change: 146241134
https://codereview.chromium.org/2678953002/

Convert class-full-of-statics CryptoTestUtils into a namespace
Merge internal change: 146240722
https://codereview.chromium.org/2671333004/

Deprecate FLAGS_quic_reloadable_flag_quic_more_conservative_retransmission_alarm
Merge internal change: 146149718
https://codereview.chromium.org/2671303003/

Replace crypto::SecureHash with calls to SHA256.
Merge internal change: 146047815
https://codereview.chromium.org/2672403003/

Make QUIC force HOL-blocking mode only apply to v36. This is basically a no-op
since neither v36 nor v37 are enabled.
Merge internal change: 146004954
https://codereview.chromium.org/2678003003/
==========

to

==========
Landing Recent QUIC changes until 5:30 PM, Feb 3, 2017 UTC-5

Flip FLAGS_quic_reloadable_flag_quic_use_event_time to true.
https://codereview.chromium.org/2680803002/

Remove path id from IV in QUIC encrytion algorithms.
Merge internal change: 146517512
https://codereview.chromium.org/2679673005/

Get rid of more va_arg methods in favor of std::vector<> arguments. Much simpler
and more readable method implementations as a result.
Merge internal change: 146383132
https://codereview.chromium.org/2671273003/

Remove two unused ProofSourceForTesting.
Merge internal change: 146291369
https://codereview.chromium.org/2676343004/

Replace va_arg version of crypto_test_utils::Message with a more modern vector
argument version.
Couple of other simplifications:
- always build a CHLO (this method was never used for anything else)
- make padding an explicit argument rather than a weird special case
- remove disabling of clang-format as things look good with this change
Merge internal change: 146270645
https://codereview.chromium.org/2678973002/

Deprecate FLAGS_quic_reloadable_flag_enable_async_get_proof
Merge internal change: 146243587
https://codereview.chromium.org/2679783003/

Deprecate FLAGS_quic_reloadable_flag_quic_only_5rto_client_side in the disabled
state, now that it's been disabled for over a month.
Merge internal change: 146241134
https://codereview.chromium.org/2678953002/

Convert class-full-of-statics CryptoTestUtils into a namespace
Merge internal change: 146240722
https://codereview.chromium.org/2671333004/

Deprecate FLAGS_quic_reloadable_flag_quic_more_conservative_retransmission_alarm
Merge internal change: 146149718
https://codereview.chromium.org/2671303003/

Replace crypto::SecureHash with calls to SHA256.
Merge internal change: 146047815
https://codereview.chromium.org/2672403003/

Make QUIC force HOL-blocking mode only apply to v36. This is basically a no-op
since neither v36 nor v37 are enabled.
Merge internal change: 146004954
https://codereview.chromium.org/2678003003/

Review-Url: https://codereview.chromium.org/2681793002
Cr-Commit-Position: refs/heads/master@{#448645}
Committed:
https://chromium.googlesource.com/chromium/src/+/f5f4cd51c92ce55d2eaadd371c6a...
==========

[commit-bot: I haz the power, 2017-02-07 16:57:50]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/f5f4cd51c92ce55d2eaadd371c6a...

[Fan Yang, 2017-02-07 22:02:33]

Patchset #3 (id:40001) has been deleted