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

Unified Diff: net/spdy/spdy_session.h

Issue 331663007: Implement PUSH_PROMISE handling in spdy_session (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 16c17a3205465d595b4867d4e4a27ea03a07d3bb..e45b7ad62eb9236639901b66de651cebcb4cc62d 100644
--- a/net/spdy/spdy_session.h
+++ b/net/spdy/spdy_session.h
@@ -537,6 +537,7 @@ class NET_EXPORT SpdySession : public BufferedSpdyFramerVisitorInterface,
SpdyStream* stream;
bool waiting_for_syn_reply;
+ bool reserved_remote;
Johnny 2014/06/17 04:01:33 Rather than having |reserved_remote| here, add an
baranovich 2014/06/17 21:33:43 Done.
};
typedef std::map<SpdyStreamId, ActiveStreamInfo> ActiveStreamMap;
@@ -611,6 +612,12 @@ class NET_EXPORT SpdySession : public BufferedSpdyFramerVisitorInterface,
// possible.
void ProcessPendingStreamRequests();
+ bool TryCreatePushStream(SpdyStreamId stream_id,
+ SpdyStreamId associated_stream_id,
+ SpdyPriority priority,
+ const SpdyHeaderBlock& headers);
+ void ActivateReservedStreamIterator(ActiveStreamMap::iterator it);
+
// Close the stream pointed to by the given iterator. Note that that
// stream may hold the last reference to the session.
void CloseActiveStreamIterator(ActiveStreamMap::iterator it, int status);
@@ -968,6 +975,7 @@ class NET_EXPORT SpdySession : public BufferedSpdyFramerVisitorInterface,
// them into a separate ActiveStreamMap, and not deliver network events to
// them?
ActiveStreamMap active_streams_;
+ size_t reserved_remote_stream_num_;
Johnny 2014/06/17 04:01:33 How about |num_reserved_remote_streams_| to match
baranovich 2014/06/17 21:33:43 Done. Deleted.
// (Bijective) map from the URL to the ID of the streams that have
// already started to be pushed by the server, but do not have

Powered by Google App Engine
This is Rietveld 408576698