Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(385)

Unified Diff: net/spdy/spdy_session.h

Issue 17004007: [SPDY] Fix SpdySession's handling of SYN_REPLY frames (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/spdy/spdy_session.h
diff --git a/net/spdy/spdy_session.h b/net/spdy/spdy_session.h
index 22b1edfbb364de1d9389bdfec0567c6f911fa0a3..35e24f2156d1e2c2d436ae4ee28fe9653a1dd59e 100644
--- a/net/spdy/spdy_session.h
+++ b/net/spdy/spdy_session.h
@@ -483,9 +483,25 @@ class NET_EXPORT SpdySession : public base::RefCounted<SpdySession>,
typedef std::deque<SpdyStreamRequest*> PendingStreamRequestQueue;
typedef std::set<SpdyStreamRequest*> PendingStreamRequestCompletionSet;
- typedef std::map<SpdyStreamId, SpdyStream*> ActiveStreamMap;
- typedef std::map<std::string, std::pair<SpdyStream*, base::TimeTicks> >
- PushedStreamMap;
+ struct ActiveStreamInfo {
+ ActiveStreamInfo();
Ryan Hamilton 2013/06/21 15:35:37 Is this constructor used explicitly, or only by of
akalin 2013/06/21 18:30:44 Just for STL, and to make sure that the member var
+ explicit ActiveStreamInfo(SpdyStream* stream);
+ ~ActiveStreamInfo();
+
+ SpdyStream* stream;
+ bool waiting_for_syn_reply;
+ };
+ typedef std::map<SpdyStreamId, ActiveStreamInfo> ActiveStreamMap;
+
+ struct PushedStreamInfo {
+ PushedStreamInfo();
+ PushedStreamInfo(SpdyStreamId stream_id, base::TimeTicks creation_time);
+ ~PushedStreamInfo();
+
+ SpdyStreamId stream_id;
+ base::TimeTicks creation_time;
+ };
+ typedef std::map<std::string, PushedStreamInfo> PushedStreamMap;
typedef std::set<SpdyStream*> CreatedStreamSet;

Powered by Google App Engine
This is Rietveld 408576698