QUIC - instrumentation for testing null QuicSpdyStream.

net/quic/quic_http_stream.h

Index: net/quic/quic_http_stream.h
diff --git a/net/quic/quic_http_stream.h b/net/quic/quic_http_stream.h
index e807a0d196396021f66cfb73c915702562ac50aa..ca5cafa3e9b7fb678d90a91a86829460ca0fa4e0 100644
--- a/net/quic/quic_http_stream.h
+++ b/net/quic/quic_http_stream.h
@@ -178,6 +178,9 @@ class NET_EXPORT_PRIVATE QuicHttpStream
   // SSLInfo from the underlying QuicSession.
   SSLInfo ssl_info_;
 
+  // TODO(rtenneti): Temporary until crbug.com/585591 is solved.
+  bool read_is_called_ = false;

[ramant (doing other things), 2016/02/11 19:45:47]

This is set to true to when we call Read(). Didn't want to rely on |stream_| is
not corrupted (ie it has become a nullptr).

Assumption: When Read() is called, ResetStream() wouldn't be called. |stream_|
will not be deleted underneath us. Or |stream_| will not be corrupted.

[eroman, 2016/02/12 01:42:46]

[optional] suggest naming this something closer to: "read_in_progress_"

[ramant (doing other things), 2016/02/12 02:47:09]

Done.

+
   base::WeakPtrFactory<QuicHttpStream> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(QuicHttpStream);