[Domain Reliabiliy: net stack] Plumb received go away from server due to connection migration.

net/quic/quic_http_stream.cc

Index: net/quic/quic_http_stream.cc
diff --git a/net/quic/quic_http_stream.cc b/net/quic/quic_http_stream.cc
index 5578bd67b718c9576e8d2cee44d945e59fef6b21..0a687ed581281873e97ab99b4d80fb86e8526fa4 100644
--- a/net/quic/quic_http_stream.cc
+++ b/net/quic/quic_http_stream.cc
@@ -46,6 +46,7 @@ QuicHttpStream::QuicHttpStream(
       closed_stream_sent_bytes_(0),
       user_buffer_len_(0),
       quic_connection_error_(QUIC_NO_ERROR),
+      port_migration_detected_(false),
       found_promise_(false),
       push_handle_(nullptr),
       weak_factory_(this) {
@@ -437,6 +438,11 @@ void QuicHttpStream::PopulateNetErrorDetails(NetErrorDetails* details) {
   details->connection_info = HttpResponseInfo::CONNECTION_INFO_QUIC1_SPDY3;
   if (was_handshake_confirmed_)
     details->quic_connection_error = quic_connection_error_;
+  if (session_) {
+    session_->PopulateNetErrorDetails(details);
+  } else {
+    details->quic_port_migration_detected = port_migration_detected_;
+  }
 }
 
 void QuicHttpStream::SetPriority(RequestPriority priority) {
@@ -514,9 +520,11 @@ void QuicHttpStream::OnCryptoHandshakeConfirmed() {
   was_handshake_confirmed_ = true;
 }
 
-void QuicHttpStream::OnSessionClosed(int error) {
+void QuicHttpStream::OnSessionClosed(int error, bool port_migration_detected) {
   Close(false);
   session_error_ = error;
+  if (!port_migration_detected_)
+    port_migration_detected_ = port_migration_detected;

[Ryan Hamilton, 2016/03/11 23:46:16]

nit: I think there's no need for the conditional since this method should only
be called once, and the member is initialized to false.

[Zhongyi Shi, 2016/03/11 23:57:23]

Done.

   session_.reset();
 }