Landing Recent QUIC changes until 12:43 PM, Nov 5, 2016 UTC+8

Landing Recent QUIC changes until 12:43 PM, Nov 5, 2016 UTC+8

There are no flags to be updated.

Implement in-flight conservation in QUIC BBR.  Protected by FLAGS_quic_allow_new_bbr.
Merge internal change: 138264727
https://codereview.chromium.org/2479103002/

Delete unused, unimplemented CheckForAddressMigration declaration.
Merge internal change: 138199910
https://codereview.chromium.org/2485553002/

Rename ReliableQuicStream to QuicStream. No behavior change, not protected.
Merge internal change: 138194450
https://codereview.chromium.org/2480183002/

Deprecate FLAGS_quic_buffer_packet_till_chlo.
Merge internal change: 138188583
https://codereview.chromium.org/2484603002/

Deprecate FLAGS_quic_reduce_sequencer_buffer_memory_life_time.
Merge internal change: 138108562
https://codereview.chromium.org/2479083002/

Update a hyperlink in quic_crypto_stream.h to point to the publicly available crypto doc.
Merge internal change: 138085207
https://codereview.chromium.org/2481123002/

Remove unused constructor of QuicPacketWriterWrapper
Merge internal change: 138079930
https://codereview.chromium.org/2481113002/

Populate connection stats with the number of BLOCKED frames sent and received.
Merge internal change: 138077328
https://codereview.chromium.org/2477273002/

Include send_algorithm_interface.h in tcp_cubic_sender_bytes_test.cc and tcp_cubic_sender_packets_test.cc.
Include quic_session.h in quic_crypto_server_stream.cc
Merge internal change: 137955026
https://codereview.chromium.org/2479073002/

Delete unused trailer_bytes_read() in quic_spdy_client_stream.h.
Merge internal change: 137703540
https://codereview.chromium.org/2484583002/

Rewrite a DLOG message to make it more readable. Currently the output is:
"Ignoring data in frame 5"
which is odd because there is no "frame 5", it's data for "stream 5".
Hopefully the rewording makes more sense.
Merge internal change: 137699688
https://codereview.chromium.org/2481703002/

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

[Fan Yang, 2016-11-08 01:44:14]

Description was changed from

==========
Landing Recent QUIC changes until 12:43 PM, Nov 5, 2016 UTC+8

There are no flags to be updated

Implement in-flight conservation in QUIC BBR.  Protected by
FLAGS_quic_allow_new_bbr.
Merge internal change: 138264727
https://codereview.chromium.org/2479103002/

Delete unused, unimplemented CheckForAddressMigration declaration.
Merge internal change: 138199910
https://codereview.chromium.org/2485553002/

Rename ReliableQuicStream to QuicStream. No behavior change, not protected.
Merge internal change: 138194450
https://codereview.chromium.org/2480183002/

Deprecate FLAGS_quic_reduce_sequencer_buffer_memory_life_time.
Merge internal change: 138108562
https://codereview.chromium.org/2479083002/

Update a hyperlink in quic_crypto_stream.h to point to the publicly available
crypto doc.
Merge internal change: 138085207
https://codereview.chromium.org/2481123002/

Remove unused constructor of QuicPacketWriterWrapper
Merge internal change: 138079930
https://codereview.chromium.org/2481113002/

Populate connection stats with the number of BLOCKED frames sent and received.
Merge internal change: 138077328
https://codereview.chromium.org/2477273002/

Include send_algorithm_interface.h in tcp_cubic_sender_bytes_test.cc and
tcp_cubic_sender_packets_test.cc.
Include quic_session.h in quic_crypto_server_stream.cc
Merge internal change: 137955026
https://codereview.chromium.org/2479073002/

Delete unused trailer_bytes_read() in quic_spdy_client_stream.h.
Merge internal change: 137703540
https://codereview.chromium.org/2484583002/

Rewrite a DLOG message to make it more readable. Currently the output is:
"Ignoring data in frame 5"
which is odd because there is no "frame 5", it's data for "stream 5".
Hopefully the rewording makes more sense.
Merge internal change: 137699688
https://codereview.chromium.org/2481703002/
==========

to

==========
Landing Recent QUIC changes until 12:43 PM, Nov 5, 2016 UTC+8

There are no flags to be updated.

Implement in-flight conservation in QUIC BBR.  Protected by
FLAGS_quic_allow_new_bbr.
Merge internal change: 138264727
https://codereview.chromium.org/2479103002/

Delete unused, unimplemented CheckForAddressMigration declaration.
Merge internal change: 138199910
https://codereview.chromium.org/2485553002/

Rename ReliableQuicStream to QuicStream. No behavior change, not protected.
Merge internal change: 138194450
https://codereview.chromium.org/2480183002/

Deprecate FLAGS_quic_buffer_packet_till_chlo.
Merge internal change: 138188583
https://codereview.chromium.org/2484603002/

Deprecate FLAGS_quic_reduce_sequencer_buffer_memory_life_time.
Merge internal change: 138108562
https://codereview.chromium.org/2479083002/

Update a hyperlink in quic_crypto_stream.h to point to the publicly available
crypto doc.
Merge internal change: 138085207
https://codereview.chromium.org/2481123002/

Remove unused constructor of QuicPacketWriterWrapper
Merge internal change: 138079930
https://codereview.chromium.org/2481113002/

Populate connection stats with the number of BLOCKED frames sent and received.
Merge internal change: 138077328
https://codereview.chromium.org/2477273002/

Include send_algorithm_interface.h in tcp_cubic_sender_bytes_test.cc and
tcp_cubic_sender_packets_test.cc.
Include quic_session.h in quic_crypto_server_stream.cc
Merge internal change: 137955026
https://codereview.chromium.org/2479073002/

Delete unused trailer_bytes_read() in quic_spdy_client_stream.h.
Merge internal change: 137703540
https://codereview.chromium.org/2484583002/

Rewrite a DLOG message to make it more readable. Currently the output is:
"Ignoring data in frame 5"
which is odd because there is no "frame 5", it's data for "stream 5".
Hopefully the rewording makes more sense.
Merge internal change: 137699688
https://codereview.chromium.org/2481703002/
==========

[Fan Yang, 2016-11-08 01:44:30]

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

[commit-bot: I haz the power, 2016-11-08 01:45:09]

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, 2016-11-08 03:14:47]

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

[commit-bot: I haz the power, 2016-11-08 03:14:48]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Fan Yang, 2016-11-08 04:09:18]

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

[Fan Yang, 2016-11-08 04:09:49]

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

[commit-bot: I haz the power, 2016-11-08 04:10:01]

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, 2016-11-08 05:05:37]

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

[commit-bot: I haz the power, 2016-11-08 05:05:38]

Dry run: This issue passed the CQ dry run.

[Fan Yang, 2016-11-08 12:02:53]

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

[commit-bot: I haz the power, 2016-11-08 12:03:04]

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, 2016-11-08 12:07:27]

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

[commit-bot: I haz the power, 2016-11-08 12:07:28]

Dry run: This issue passed the CQ dry run.

[Ryan Hamilton, 2016-11-08 15:10:00]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2016-11-08 15:10:00]

lgtm

[commit-bot: I haz the power, 2016-11-08 15:10:16]

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, 2016-11-08 15:26:10]

Description was changed from

==========
Landing Recent QUIC changes until 12:43 PM, Nov 5, 2016 UTC+8

There are no flags to be updated.

Implement in-flight conservation in QUIC BBR.  Protected by
FLAGS_quic_allow_new_bbr.
Merge internal change: 138264727
https://codereview.chromium.org/2479103002/

Delete unused, unimplemented CheckForAddressMigration declaration.
Merge internal change: 138199910
https://codereview.chromium.org/2485553002/

Rename ReliableQuicStream to QuicStream. No behavior change, not protected.
Merge internal change: 138194450
https://codereview.chromium.org/2480183002/

Deprecate FLAGS_quic_buffer_packet_till_chlo.
Merge internal change: 138188583
https://codereview.chromium.org/2484603002/

Deprecate FLAGS_quic_reduce_sequencer_buffer_memory_life_time.
Merge internal change: 138108562
https://codereview.chromium.org/2479083002/

Update a hyperlink in quic_crypto_stream.h to point to the publicly available
crypto doc.
Merge internal change: 138085207
https://codereview.chromium.org/2481123002/

Remove unused constructor of QuicPacketWriterWrapper
Merge internal change: 138079930
https://codereview.chromium.org/2481113002/

Populate connection stats with the number of BLOCKED frames sent and received.
Merge internal change: 138077328
https://codereview.chromium.org/2477273002/

Include send_algorithm_interface.h in tcp_cubic_sender_bytes_test.cc and
tcp_cubic_sender_packets_test.cc.
Include quic_session.h in quic_crypto_server_stream.cc
Merge internal change: 137955026
https://codereview.chromium.org/2479073002/

Delete unused trailer_bytes_read() in quic_spdy_client_stream.h.
Merge internal change: 137703540
https://codereview.chromium.org/2484583002/

Rewrite a DLOG message to make it more readable. Currently the output is:
"Ignoring data in frame 5"
which is odd because there is no "frame 5", it's data for "stream 5".
Hopefully the rewording makes more sense.
Merge internal change: 137699688
https://codereview.chromium.org/2481703002/
==========

to

==========
Landing Recent QUIC changes until 12:43 PM, Nov 5, 2016 UTC+8

There are no flags to be updated.

Implement in-flight conservation in QUIC BBR.  Protected by
FLAGS_quic_allow_new_bbr.
Merge internal change: 138264727
https://codereview.chromium.org/2479103002/

Delete unused, unimplemented CheckForAddressMigration declaration.
Merge internal change: 138199910
https://codereview.chromium.org/2485553002/

Rename ReliableQuicStream to QuicStream. No behavior change, not protected.
Merge internal change: 138194450
https://codereview.chromium.org/2480183002/

Deprecate FLAGS_quic_buffer_packet_till_chlo.
Merge internal change: 138188583
https://codereview.chromium.org/2484603002/

Deprecate FLAGS_quic_reduce_sequencer_buffer_memory_life_time.
Merge internal change: 138108562
https://codereview.chromium.org/2479083002/

Update a hyperlink in quic_crypto_stream.h to point to the publicly available
crypto doc.
Merge internal change: 138085207
https://codereview.chromium.org/2481123002/

Remove unused constructor of QuicPacketWriterWrapper
Merge internal change: 138079930
https://codereview.chromium.org/2481113002/

Populate connection stats with the number of BLOCKED frames sent and received.
Merge internal change: 138077328
https://codereview.chromium.org/2477273002/

Include send_algorithm_interface.h in tcp_cubic_sender_bytes_test.cc and
tcp_cubic_sender_packets_test.cc.
Include quic_session.h in quic_crypto_server_stream.cc
Merge internal change: 137955026
https://codereview.chromium.org/2479073002/

Delete unused trailer_bytes_read() in quic_spdy_client_stream.h.
Merge internal change: 137703540
https://codereview.chromium.org/2484583002/

Rewrite a DLOG message to make it more readable. Currently the output is:
"Ignoring data in frame 5"
which is odd because there is no "frame 5", it's data for "stream 5".
Hopefully the rewording makes more sense.
Merge internal change: 137699688
https://codereview.chromium.org/2481703002/
==========

[commit-bot: I haz the power, 2016-11-08 15:26:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-08 15:28:28]

Description was changed from

==========
Landing Recent QUIC changes until 12:43 PM, Nov 5, 2016 UTC+8

There are no flags to be updated.

Implement in-flight conservation in QUIC BBR.  Protected by
FLAGS_quic_allow_new_bbr.
Merge internal change: 138264727
https://codereview.chromium.org/2479103002/

Delete unused, unimplemented CheckForAddressMigration declaration.
Merge internal change: 138199910
https://codereview.chromium.org/2485553002/

Rename ReliableQuicStream to QuicStream. No behavior change, not protected.
Merge internal change: 138194450
https://codereview.chromium.org/2480183002/

Deprecate FLAGS_quic_buffer_packet_till_chlo.
Merge internal change: 138188583
https://codereview.chromium.org/2484603002/

Deprecate FLAGS_quic_reduce_sequencer_buffer_memory_life_time.
Merge internal change: 138108562
https://codereview.chromium.org/2479083002/

Update a hyperlink in quic_crypto_stream.h to point to the publicly available
crypto doc.
Merge internal change: 138085207
https://codereview.chromium.org/2481123002/

Remove unused constructor of QuicPacketWriterWrapper
Merge internal change: 138079930
https://codereview.chromium.org/2481113002/

Populate connection stats with the number of BLOCKED frames sent and received.
Merge internal change: 138077328
https://codereview.chromium.org/2477273002/

Include send_algorithm_interface.h in tcp_cubic_sender_bytes_test.cc and
tcp_cubic_sender_packets_test.cc.
Include quic_session.h in quic_crypto_server_stream.cc
Merge internal change: 137955026
https://codereview.chromium.org/2479073002/

Delete unused trailer_bytes_read() in quic_spdy_client_stream.h.
Merge internal change: 137703540
https://codereview.chromium.org/2484583002/

Rewrite a DLOG message to make it more readable. Currently the output is:
"Ignoring data in frame 5"
which is odd because there is no "frame 5", it's data for "stream 5".
Hopefully the rewording makes more sense.
Merge internal change: 137699688
https://codereview.chromium.org/2481703002/
==========

to

==========
Landing Recent QUIC changes until 12:43 PM, Nov 5, 2016 UTC+8

There are no flags to be updated.

Implement in-flight conservation in QUIC BBR.  Protected by
FLAGS_quic_allow_new_bbr.
Merge internal change: 138264727
https://codereview.chromium.org/2479103002/

Delete unused, unimplemented CheckForAddressMigration declaration.
Merge internal change: 138199910
https://codereview.chromium.org/2485553002/

Rename ReliableQuicStream to QuicStream. No behavior change, not protected.
Merge internal change: 138194450
https://codereview.chromium.org/2480183002/

Deprecate FLAGS_quic_buffer_packet_till_chlo.
Merge internal change: 138188583
https://codereview.chromium.org/2484603002/

Deprecate FLAGS_quic_reduce_sequencer_buffer_memory_life_time.
Merge internal change: 138108562
https://codereview.chromium.org/2479083002/

Update a hyperlink in quic_crypto_stream.h to point to the publicly available
crypto doc.
Merge internal change: 138085207
https://codereview.chromium.org/2481123002/

Remove unused constructor of QuicPacketWriterWrapper
Merge internal change: 138079930
https://codereview.chromium.org/2481113002/

Populate connection stats with the number of BLOCKED frames sent and received.
Merge internal change: 138077328
https://codereview.chromium.org/2477273002/

Include send_algorithm_interface.h in tcp_cubic_sender_bytes_test.cc and
tcp_cubic_sender_packets_test.cc.
Include quic_session.h in quic_crypto_server_stream.cc
Merge internal change: 137955026
https://codereview.chromium.org/2479073002/

Delete unused trailer_bytes_read() in quic_spdy_client_stream.h.
Merge internal change: 137703540
https://codereview.chromium.org/2484583002/

Rewrite a DLOG message to make it more readable. Currently the output is:
"Ignoring data in frame 5"
which is odd because there is no "frame 5", it's data for "stream 5".
Hopefully the rewording makes more sense.
Merge internal change: 137699688
https://codereview.chromium.org/2481703002/

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

[commit-bot: I haz the power, 2016-11-08 15:28:29]

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