Land Recent QUIC changes.

Land Recent QUIC changes.

Added a DCHECK that ProcessRawData will not be called with zero data length.

Merge internal change: 52068687

Removed reinterpret_cast<QuicFecBuilderInterface*>. Minor comment fix.

Merge internal change: 52051673

Replace QuicConnectionVisitorInterface::OnPacket with OnStreamFrames

Merge internal change: 52049491

Remove unused OnAck method from QuicConnectionVisitorInterface.

This is a bit of an abstraction violation, since nothing above
the Connection knows about packets.  The AckNotifier interface
provides the "right" abstraction.

Merge internal change: 52027657

Add state to session to indicate that a crypto stream is blocked, so
that we can properly set the IS_HANDSHAKE argument when asking the
congestion manager if we CanWrite().

Merge internal change: 51983405

Break out the tracking of sent packet from QuicConnection to a new
QuicSentPacketManager class.

Merge internal change: 51969314

Delay the RTO every time an ack is received, per TCP RTO spec.

Merge internal change: 51954312

Sending ConnectionClose frames via the PacketGenerator so they get
properly queued if they cannot be sent immediately.  Prevents multiple
different connection close packets from being sent, a potential
infinite loop, and from dropping connection close packets which can't
be sent due to a write block from being dropped entirely.

Merge internal change: 51890780

Increse the default max_tcp_congestion_window to 100 from 50, now that
the RTO has been increased to the TCP standard of 60 seconds.

Merge internal change: 51854728

Added large (1 Mb) POST tests.

Verified that without the fix from cl/51829697 these tests timeout due
to receiver cwnd dropping to zero after sending ACKs.

Merge internal change: 51854636

Refactored and replaced SendStreamAndNotifyWhenAck() to work with
SendvStreamData().

Merge internal change: 51848330

Remove unused client2 scoped_ptrs from end_to_end_tests

Merge internal change: 51835589

R=rch@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223996

[ramant (doing other things), 2013-09-17 22:25:10]

Hi Ryan,
  This CL has all the CL's you have LGTM'ed after what was already merged in CL
https://codereview.chromium.org/23597045/.

  This CL doesn't have "Track the retransmission history of a QUIC packet so
that if a previous" (Merge internal change: 52125881) and "Change Retransmission
to TransmissionType and split IS_RETRANSMISSION" (Merge internal change:
52187292).


thanks
raman

[Ryan Hamilton, 2013-09-18 03:39:26]

lgtm since i have reviewd all the individual changes.

[commit-bot: I haz the power, 2013-09-18 15:44:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/23691073/8001

[commit-bot: I haz the power, 2013-09-18 16:00:38]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-09-18 16:50:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/23691073/8001

[commit-bot: I haz the power, 2013-09-18 17:00:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/23691073/26001

[commit-bot: I haz the power, 2013-09-18 17:05:09]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_clan...

[commit-bot: I haz the power, 2013-09-18 18:07:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/23691073/41001

[commit-bot: I haz the power, 2013-09-18 23:54:49]

Change committed as 223996