Adds socket param to OnReadError in QuicPacketReader's Visitor interface.

Adds socket param to OnReadError in QuicPacketReader's Visitor interface.
(Breaking out connection migration CL https://codereview.chromium.org/1327923002/.)

R=rch@chromium.org,pauljensen@chromium.org
BUG=

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

[Jana, 2015-12-22 20:53:25]

Description was changed from

==========
Adds socket param to OnReadError in QuicPacketReader's Visitor interface.
(Breaking out connection migration CL
https://codereview.chromium.org/1327923002/.)

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

to

==========
Adds socket param to OnReadError in QuicPacketReader's Visitor interface.
(Breaking out connection migration CL
https://codereview.chromium.org/1327923002/.)

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

[Jana, 2015-12-22 20:53:25]

jri@chromium.org changed reviewers:
+ pauljensen@chromium.org

[Jana, 2015-12-22 20:53:44]

Description was changed from

==========
Adds socket param to OnReadError in QuicPacketReader's Visitor interface.
(Breaking out connection migration CL
https://codereview.chromium.org/1327923002/.)

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

to

==========
Adds socket param to OnReadError in QuicPacketReader's Visitor interface.
(Breaking out connection migration CL
https://codereview.chromium.org/1327923002/.)

R=rch@chromium.org,pauljensen@chromium.org
BUG=
==========

[Ryan Hamilton, 2015-12-22 22:03:45]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2015-12-22 22:03:45]

lgtm

https://codereview.chromium.org/1543903002/diff/1/net/quic/quic_chromium_clie...
File net/quic/quic_chromium_client_session.cc (right):

https://codereview.chromium.org/1543903002/diff/1/net/quic/quic_chromium_clie...
net/quic/quic_chromium_client_session.cc:917: const DatagramClientSocket*
socket) {
Typically, we would do /*socket*/ here to indicate that the argument is ignored,
but since that's going to change in the follow up CL, I think it's fine to leave
it this way.

[commit-bot: I haz the power, 2015-12-22 22:04:08]

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

[commit-bot: I haz the power, 2015-12-22 23:28:19]

Description was changed from

==========
Adds socket param to OnReadError in QuicPacketReader's Visitor interface.
(Breaking out connection migration CL
https://codereview.chromium.org/1327923002/.)

R=rch@chromium.org,pauljensen@chromium.org
BUG=
==========

to

==========
Adds socket param to OnReadError in QuicPacketReader's Visitor interface.
(Breaking out connection migration CL
https://codereview.chromium.org/1327923002/.)

R=rch@chromium.org,pauljensen@chromium.org
BUG=
==========

[commit-bot: I haz the power, 2015-12-22 23:28:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-22 23:30:01]

Description was changed from

==========
Adds socket param to OnReadError in QuicPacketReader's Visitor interface.
(Breaking out connection migration CL
https://codereview.chromium.org/1327923002/.)

R=rch@chromium.org,pauljensen@chromium.org
BUG=
==========

to

==========
Adds socket param to OnReadError in QuicPacketReader's Visitor interface.
(Breaking out connection migration CL
https://codereview.chromium.org/1327923002/.)

R=rch@chromium.org,pauljensen@chromium.org
BUG=

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

[commit-bot: I haz the power, 2015-12-22 23:30:01]

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