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

net/quic/quic_chromium_client_session.cc

Index: net/quic/quic_chromium_client_session.cc
diff --git a/net/quic/quic_chromium_client_session.cc b/net/quic/quic_chromium_client_session.cc
index 18aabbc317c4ff1afbd7758ebf083f368027136f..5c9a75115dd11216140b40c9e9ff234c693e1963 100644
--- a/net/quic/quic_chromium_client_session.cc
+++ b/net/quic/quic_chromium_client_session.cc
@@ -912,7 +912,9 @@ QuicChromiumClientSession::GetWeakPtr() {
   return weak_factory_.GetWeakPtr();
 }
 
-void QuicChromiumClientSession::OnReadError(int result) {
+void QuicChromiumClientSession::OnReadError(
+    int result,
+    const DatagramClientSocket* socket) {

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

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.

   DVLOG(1) << "Closing session on read error: " << result;
   UMA_HISTOGRAM_SPARSE_SLOWLY("Net.QuicSession.ReadError", -result);
   NotifyFactoryOfSessionGoingAway();