Landing Recent QUIC Changes.

Landing Recent QUIC Changes.

Ensure the QuicConnection's writer is not write blocked before trying to
write another packet.

Fix for Chrome crash bug 409191.

Merge internal change: 74635422

BUG=409191
Nest a QUIC SerializedPacket inside a QUIC QueuedPacket.

Merge internal change: 74239145
https://codereview.chromium.org/509203003/


Remove PacketType from QUIC because the QUEUED type is not used and the
other types can be reduced to a bool.

Merge internal change: 74148481
https://codereview.chromium.org/515003003/


Don't send a QUIC SCUP message until after handshake confirmed.

A server config update was occasionally being sent before the crypto
handshake was complete, causing the client to close the connection.

Merge internal change: 74132773
https://codereview.chromium.org/516713002/


Change TransmissionInfo's all_transmissions SequenceNumberSet* to
default to NULL, and only be present if there are multiple transmissions.

Estimated to save ~2% of CPU.

Merge internal change: 74076012
https://codereview.chromium.org/509073004/


Log the quic version for internal server tracing.
Not used in production.

Merge internal change: 74069715
https://codereview.chromium.org/514043002/


Optimize QuicUnackedPacketMap by changing from a LinkedHashMap to a
deque.

Estimated to save ~3% of CPU based on pprof profiling of 100 large gets
on tools/quic/end_to_end_test.cc.

Merge internal change: 74054196
https://codereview.chromium.org/514033002/


Unit test for empty packet closes QUIC connection bug.

Merge internal change: 74041239
https://codereview.chromium.org/495423011/


Log the QUIC transmission type in QUIC internal server trace visitor.
Not used in production.

Merge internal change: 73895739
https://codereview.chromium.org/514023002/


Not used in production. Log whether a QUIC packet sent was a crypto
packet or not for internal server side tracing.

Merge internal change: 73894063
https://codereview.chromium.org/515003002/


Fix a QUIC bug in which PING frames were not being ACK'd.

Merge internal change: 73837644
https://codereview.chromium.org/512933005/

R=rch@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/27f3f1894f09a96112df50fddd32895b4677f949

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

[ramant (doing other things), 2014-08-28 22:15:59]

On 2014/08/28 17:53:55, ramant wrote:

Hi Ryan,
  This CL is a merge of all the CL's you have approved. Would appreciate if you
could take a look.

thanks very much
raman

[Ryan Hamilton, 2014-08-28 22:22:12]

lgtm

[ramant (doing other things), 2014-08-28 22:25:56]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2014-08-28 22:27:25]

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

[commit-bot: I haz the power, 2014-08-28 23:39:57]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-29 00:54:36]

Committed patchset #1 (id:1) as 27f3f1894f09a96112df50fddd32895b4677f949

[ramant (doing other things), 2014-08-29 23:36:26]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/523813003/ by rtenneti@chromium.org.

The reason for reverting is: Reverting to see if it fixes the following issue.
Crash rate has gone up by 10 times.

BUG=409191.

[ramant (doing other things), 2014-09-02 22:29:17]

rtenneti@chromium.org changed reviewers:
+ ianswett@google.com

[ramant (doing other things), 2014-09-02 22:29:18]

Hi Ryan and Ian,
  Merged the changes from internal CL: 74635422. Will merge changes before that
CL in another CL.

  PTAL.

thanks,
raman

[Ryan Hamilton, 2014-09-02 22:32:08]

lgtm

[ramant (doing other things), 2014-09-02 22:36:44]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2014-09-02 22:37:51]

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

[commit-bot: I haz the power, 2014-09-02 23:54:15]

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

[commit-bot: I haz the power, 2014-09-02 23:54:16]

Failed to commit the patch.

[ramant (doing other things), 2014-09-03 00:04:04]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2014-09-03 00:06:23]

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

[commit-bot: I haz the power, 2014-09-03 00:46:18]

Committed patchset #3 (id:40001) as 93ae0005c8e7d8386d37f010ecc4b2001541ea07

[commit-bot: I haz the power, 2014-09-10 03:04:32]

Patchset 3 (id:??) landed as
https://crrev.com/33089992bfb2436986b9be2ddd76ace321430de8
Cr-Commit-Position: refs/heads/master@{#292527}

[commit-bot: I haz the power, 2014-09-10 03:22:13]

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