Landing Recent QUIC changes until 01/28/2016 18:41 UTC

Landing Recent QUIC changes until 01/28/2016 18:41 UTC

use std::unordered_map and std::unordered_set in QUIC code instead of
hash_map and hash_set.

Merge internal change: 113278032
https://codereview.chromium.org/1663703003/

Postpone checking peer address change after the packet is successfully
decrypted. Protected behind FLAGS_check_peer_address_change_after_decryption.

Merge internal change: 113259990
https://codereview.chromium.org/1663493003/

Fixing up includes for common_cert_set.cc

common_cert_set_1.cc and 2.cc need to handle size_t correctly

Merge internal change: 113259867
https://codereview.chromium.org/1661883002/

Make QUIC's SerializedPacket contain QuicFrames, rather than a pointer
to QuicFrames.  No functional change.

Estimated to save ~0.8% CPU.

Merge internal change: 113222997
https://codereview.chromium.org/1666553002/

factor out path id + packet number packing code from the QUIC aead code
into QuicUtils::PackPathIdAndPacketNumber to reduce code duplication
(which is doubled in Chromium because of NSS & OpenSSL versions of this
code).

Merge internal change: 113205205
https://codereview.chromium.org/1660253003/

Add FLAGS_quic_crypto_server_config_default_has_chacha20, to allow
internal server to pretend it doesn't support ChaCha20.

Merge internal change: 113179972
https://codereview.chromium.org/1665693002/

Remove a test-only SerializedPacket constructor from QUIC.

n/a (Test-only change)

Merge internal change: 113125505
https://codereview.chromium.org/1661863002/

Fix a bug in QUIC's Slow Start Large Reduction connection option that
increased packet loss.  Small functional change.

Merge internal change: 113110142
https://codereview.chromium.org/1660143003/

Make a SerializedPacket a member variable of QuicPacketCreator to
replace the corresponding fields.  No functional change, not flag
protected.

Merge internal change: 113108401
https://codereview.chromium.org/1658393003/

Deprecate FLAGS_quic_general_loss_algorithm.

Merge internal change: 113076102
https://codereview.chromium.org/1667513002/

Move QuicPacketCreator::SerializeAllFrames into QuicPacketCreatorPeer.
 No functional change.

Merge internal change: 113072084
https://codereview.chromium.org/1658373002/

R=rch@chromium.org

Committed: https://crrev.com/f85706db6ab7f2e5a5feeb89a8cd895fcb176f0b
Cr-Commit-Position: refs/heads/master@{#373440}

[ramant (doing other things), 2016-02-03 21:21:58]

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

[commit-bot: I haz the power, 2016-02-03 21:23:08]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1660593004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1660593004/1

[commit-bot: I haz the power, 2016-02-03 22:38:16]

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

[commit-bot: I haz the power, 2016-02-03 22:38:16]

Dry run: This issue passed the CQ dry run.

[Ryan Hamilton, 2016-02-04 00:15:13]

lgtm

[ramant (doing other things), 2016-02-04 02:34:36]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2016-02-04 02:35:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1660593004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1660593004/1

[commit-bot: I haz the power, 2016-02-04 02:43:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-04 02:44:35]

Description was changed from

==========
Landing Recent QUIC changes until 01/28/2016 18:41 UTC

use std::unordered_map and std::unordered_set in QUIC code instead of
hash_map and hash_set.

Merge internal change: 113278032
https://codereview.chromium.org/1663703003/


Postpone checking peer address change after the packet is successfully
decrypted. Protected behind FLAGS_check_peer_address_change_after_decryption.

Merge internal change: 113259990
https://codereview.chromium.org/1663493003/


Fixing up includes for common_cert_set.cc

common_cert_set_1.cc and 2.cc need to handle size_t correctly

Merge internal change: 113259867
https://codereview.chromium.org/1661883002/


Make QUIC's SerializedPacket contain QuicFrames, rather than a pointer
to QuicFrames.  No functional change.

Estimated to save ~0.8% CPU.

Merge internal change: 113222997
https://codereview.chromium.org/1666553002/


factor out path id + packet number packing code from the QUIC aead code
into QuicUtils::PackPathIdAndPacketNumber to reduce code duplication
(which is doubled in Chromium because of NSS & OpenSSL versions of this
code).

Merge internal change: 113205205
https://codereview.chromium.org/1660253003/


Add FLAGS_quic_crypto_server_config_default_has_chacha20, to allow
internal server to pretend it doesn't support ChaCha20.

Merge internal change: 113179972
https://codereview.chromium.org/1665693002/


Remove a test-only SerializedPacket constructor from QUIC.

n/a (Test-only change)

Merge internal change: 113125505
https://codereview.chromium.org/1661863002/


Fix a bug in QUIC's Slow Start Large Reduction connection option that
increased packet loss.  Small functional change.

Merge internal change: 113110142
https://codereview.chromium.org/1660143003/


Make a SerializedPacket a member variable of QuicPacketCreator to
replace the corresponding fields.  No functional change, not flag
protected.

Merge internal change: 113108401
https://codereview.chromium.org/1658393003/


Deprecate FLAGS_quic_general_loss_algorithm.

Merge internal change: 113076102
https://codereview.chromium.org/1667513002/


Move QuicPacketCreator::SerializeAllFrames into QuicPacketCreatorPeer.
 No functional change.

Merge internal change: 113072084
https://codereview.chromium.org/1658373002/

R=rch@chromium.org
==========

to

==========
Landing Recent QUIC changes until 01/28/2016 18:41 UTC

use std::unordered_map and std::unordered_set in QUIC code instead of
hash_map and hash_set.

Merge internal change: 113278032
https://codereview.chromium.org/1663703003/

Postpone checking peer address change after the packet is successfully
decrypted. Protected behind FLAGS_check_peer_address_change_after_decryption.

Merge internal change: 113259990
https://codereview.chromium.org/1663493003/

Fixing up includes for common_cert_set.cc

common_cert_set_1.cc and 2.cc need to handle size_t correctly

Merge internal change: 113259867
https://codereview.chromium.org/1661883002/

Make QUIC's SerializedPacket contain QuicFrames, rather than a pointer
to QuicFrames.  No functional change.

Estimated to save ~0.8% CPU.

Merge internal change: 113222997
https://codereview.chromium.org/1666553002/

factor out path id + packet number packing code from the QUIC aead code
into QuicUtils::PackPathIdAndPacketNumber to reduce code duplication
(which is doubled in Chromium because of NSS & OpenSSL versions of this
code).

Merge internal change: 113205205
https://codereview.chromium.org/1660253003/

Add FLAGS_quic_crypto_server_config_default_has_chacha20, to allow
internal server to pretend it doesn't support ChaCha20.

Merge internal change: 113179972
https://codereview.chromium.org/1665693002/

Remove a test-only SerializedPacket constructor from QUIC.

n/a (Test-only change)

Merge internal change: 113125505
https://codereview.chromium.org/1661863002/

Fix a bug in QUIC's Slow Start Large Reduction connection option that
increased packet loss.  Small functional change.

Merge internal change: 113110142
https://codereview.chromium.org/1660143003/

Make a SerializedPacket a member variable of QuicPacketCreator to
replace the corresponding fields.  No functional change, not flag
protected.

Merge internal change: 113108401
https://codereview.chromium.org/1658393003/

Deprecate FLAGS_quic_general_loss_algorithm.

Merge internal change: 113076102
https://codereview.chromium.org/1667513002/

Move QuicPacketCreator::SerializeAllFrames into QuicPacketCreatorPeer.
 No functional change.

Merge internal change: 113072084
https://codereview.chromium.org/1658373002/

R=rch@chromium.org

Committed: https://crrev.com/f85706db6ab7f2e5a5feeb89a8cd895fcb176f0b
Cr-Commit-Position: refs/heads/master@{#373440}
==========

[commit-bot: I haz the power, 2016-02-04 02:44:36]

Patchset 1 (id:??) landed as
https://crrev.com/f85706db6ab7f2e5a5feeb89a8cd895fcb176f0b
Cr-Commit-Position: refs/heads/master@{#373440}