Implements migration of a QUIC connection to a different destination address if specified by the se…

net/quic/quic_stream_factory.h

Index: net/quic/quic_stream_factory.h
diff --git a/net/quic/quic_stream_factory.h b/net/quic/quic_stream_factory.h
index 2f014d1242105c7ca6459381dd2b78bda8c7f588..21c4411f055fd43e524f91092ca285dad994f591 100644
--- a/net/quic/quic_stream_factory.h
+++ b/net/quic/quic_stream_factory.h
@@ -296,13 +296,14 @@ class NET_EXPORT_PRIVATE QuicStreamFactory
                                  MigrationCause migration_cause,
                                  scoped_refptr<StringIOBuffer> packet);
 
-  // Method that migrates |session| over to using |new_network|. If
-  // not null, |packet| is sent on the new network, else a PING frame
-  // is sent. Returns ERR_QUIC_PROTOCOL_ERROR if migration fails.
-  void MigrateSessionToNetwork(QuicChromiumClientSession* session,
-                               NetworkChangeNotifier::NetworkHandle new_network,
-                               const BoundNetLog& bound_net_log,
-                               scoped_refptr<StringIOBuffer> packet);
+  // Method that migrates |session| over to using |peer_address| and
+  // |new_network|. If not null, |packet| is sent on the new network,
+  // else a PING frame is sent.
+  void MigrateSessionToNewSocket(QuicChromiumClientSession* session,
+                                 IPEndPoint peer_address,
+                                 NetworkChangeNotifier::NetworkHandle network,

[Ryan Hamilton, 2016/07/06 19:29:37]

Can you document what happens when |network| is kInvalid?

Alternatively, consider making this method private and have two wrappers:
  MigrateSessionToNewNetwork
  MigrateSessionToNewPeerAddress

*shrug*

[Jana, 2016/07/12 22:34:10]

Good suggestion. Done.

+                                 const BoundNetLog& bound_net_log,
+                                 scoped_refptr<StringIOBuffer> packet);
 
   // NetworkChangeNotifier::IPAddressObserver methods: