Do not invoke QuicHttpStream::DoCallback when in DoLoop

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 39784fe2c5828246f32c81a4cd7b8e699e7d8134..fb2b798eac890bc90b2897ed477ee79ff9fe28db 100644
--- a/net/quic/quic_http_stream.cc
+++ b/net/quic/quic_http_stream.cc
@@ -510,6 +510,10 @@ void QuicHttpStream::OnClose() {
 
   quic_connection_error_ = stream_->connection_error();
   ResetStream();
+  if (in_loop_) {
+    // If already in DoLoop(), |callback_| will be handled when DoLoop() exits.
+    return;
+  }

[Ryan Hamilton, 2016/07/21 22:29:33]

Do you know what state in DoLoop we're in when this happens?

[xunjieli, 2016/07/22 13:21:22]

According to the crash log on crbug/629043, we are in DoSendBody().

   if (!callback_.is_null()) {
     DoCallback(response_status_);
   }