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

net/quic/quic_chromium_client_session.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/quic/quic_chromium_client_session.h"
 
 #include "base/callback_helpers.h"
 #include "base/location.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/metrics/sparse_histogram.h"
@@ skipping 894 matching lines @@
   dict->Set("aliases", alias_list.Pass());
 
   return dict.Pass();
 }
 
 base::WeakPtr<QuicChromiumClientSession>
 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();
   CloseSessionOnErrorInner(result, QUIC_PACKET_READ_ERROR);
   NotifyFactoryOfSessionClosedLater();
 }
 
 bool QuicChromiumClientSession::OnPacket(const QuicEncryptedPacket& packet,
                                          IPEndPoint local_address,
                                          IPEndPoint peer_address) {
@@ skipping 49 matching lines @@
     return;
 
   // TODO(rch): re-enable this code once beta is cut.
   //  if (stream_factory_)
   //    stream_factory_->OnSessionConnectTimeout(this);
   //  CloseAllStreams(ERR_QUIC_HANDSHAKE_FAILED);
   //  DCHECK_EQ(0u, GetNumOpenOutgoingStreams());
 }
 
 }  // namespace net