Change WeakPtr<SpdyStream> to raw pointer in SpdyHttpStream.

net/spdy/spdy_http_stream.h

Index: net/spdy/spdy_http_stream.h
diff --git a/net/spdy/spdy_http_stream.h b/net/spdy/spdy_http_stream.h
index 0796e1a6905f6f57ec2a4456795ced6492da7185..c996a53e96a1e108b68072203158d61a0da1a35d 100644
--- a/net/spdy/spdy_http_stream.h
+++ b/net/spdy/spdy_http_stream.h
@@ -36,7 +36,7 @@ class NET_EXPORT_PRIVATE SpdyHttpStream : public SpdyStream::Delegate,
   SpdyHttpStream(const base::WeakPtr<SpdySession>& spdy_session, bool direct);
   ~SpdyHttpStream() override;
 
-  SpdyStream* stream() { return stream_.get(); }
+  SpdyStream* stream() { return stream_; }
 
   // Cancels any callbacks from being invoked and deletes the stream.
   void Cancel();
@@ -128,7 +128,7 @@ class NET_EXPORT_PRIVATE SpdyHttpStream : public SpdyStream::Delegate,
   const base::WeakPtr<SpdySession> spdy_session_;
   bool is_reused_;
   SpdyStreamRequest stream_request_;
-  base::WeakPtr<SpdyStream> stream_;
+  SpdyStream* stream_;

[xunjieli, 2017/01/20 02:57:20]

Can you add a comment here on who owns |stream_| and when the pointer is
guaranteed to be valid (e.g. after InitializeStream() and before
Delegate::OnClose()).

[Bence, 2017/01/20 15:53:13]

Done.  Also added a comment for |stream_closed_| once we are here.

 
   bool stream_closed_;